developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Opened 9 years ago

Closed 9 years ago

#1424 closed bug (fixed)

testapp fails on utils:jimagemodifier when having a reverse proxy

Reported by: bricet Owned by: laurentj
Priority: normal Milestone: Jelix 1.3
Component: app:testapp Version: trunk
Severity: minor Keywords: testapp proxy
Cc: Blocked By:
Blocking: Documentation needed: no
Hosting Provider: Php version:

Description

On my dev machine, I use nginx as a reverse proxy on Apache.

Nginx listens on port 80 and Apache on port 1080.

I have some fails on testapp because we use $_SERVERHTTP_HOST? which contains Apache's port :

Fail: /home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php -> UTjimagemodifier -> testGet -> Equal expectation fails at character 16 with [http://127.0.0.1/testapp/www/cache/images/cee9525fd683b795408902bab4229989.png] and [http://127.0.0.1:1080/testapp/www/cache/images/cee9525fd683b795408902bab4229989.png] at [/home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php line 34]
Fail: /home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php -> UTjimagemodifier -> testGetOmo -> Equal expectation fails at character 16 with [http://127.0.0.1/testapp/www/cache/images/0c1572bf62e89658af638dd7615aad0e.png] and [http://127.0.0.1:1080/testapp/www/cache/images/0c1572bf62e89658af638dd7615aad0e.png] at [/home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php line 54]
Fail: /home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php -> UTjimagemodifier -> testGetOmo2 -> Equal expectation fails at character 16 with [http://127.0.0.1/testapp/www/cache/images/c3d31f397feea33aded3ca63dad4b25d.png] and [http://127.0.0.1:1080/testapp/www/cache/images/c3d31f397feea33aded3ca63dad4b25d.png] at [/home/brice/dev/lib_jelix/fork-rde/git/compiled/dev/testapp/modules/jelix_tests/tests/utils.jimagemodifier.html_cli.php line 73]

I have a workaround which should soon be pull requested (enjoying our brand new toy ;) )

Change History (2)

comment:1 Changed 9 years ago by bricet

  • review set to review?

Done : https://github.com/jelix/jelix/pull/2

May be this problem may occur somewhere else in the code, but this does not lead to others troubles in my current testapp ...

comment:2 Changed 9 years ago by laurentj

  • Milestone set to Jelix 1.3
  • Resolution set to fixed
  • review changed from review? to review+
  • Status changed from new to closed
Note: See TracTickets for help on using tickets.