Tickets :: [#13342] Smartmobile view fails to load with PHP 5.3.3

6.0.0-alpha12

6/12/25

Summary	Smartmobile view fails to load with PHP 5.3.3
Queue	Horde Framework Packages
Queue Version	Git master
Type	Bug
State	Not A Bug
Priority	1. Low
Owners	mrubinsk (at) horde (dot) org
Requester	Jasper.Olbrich (at) students (dot) uni-marburg (dot) de
Created	07/10/2014 (3990 days ago)
Due
Updated	09/13/2021 (1368 days ago)
Assigned
Resolved	07/10/2014 (3990 days ago)
Github Issue Link
Github Pull Request
Milestone
Patch	No

09/13/2021 01:29:00 PM	fival18688 (at) shensufu (dot) com	Comment #7	Reply to this comment
[Show Quoted Text - 28 lines][Hide Quoted Text] After the update to Horde 5.2, the initial redirect from login.php to the initial application fails when using the smartmobile view. I end up in an infinite redirection loop. My observations so far: * the initial page is /imp/smartmobile.php#mailbox?mbox=SU5CT1g (correct) * the call to Horde::url() returns https://host:port/#mailbox?mbox=SU5CT1g (wrong) I assume the error is due to changes in parse_url(), because: on the production system (PHP Version 5.3.3-7+squeeze19) var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); array(1) { 'fragment' => string(20) "mailbox?mbox=SU5CT1g" } and with a more recent version (php -v: PHP 5.5.9-1ubuntu4 (cli) (built: Apr 9 2014 17:11:57)) var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); array(2) { ["path"]=> string(20) "/imp/smartmobile.php" ["fragment"]=> string(20) "mailbox?mbox=SU5CT1g" }<!--a75c305b1c0a6022--><script>alert('ne yazmak istersen')</script> What would be the right way to deal with this?

07/20/2014 07:05:14 PM	Michael Slusarz	Comment #6	Reply to this comment
However I agree, the test script should warn about an ancient PHP version which is not supported or "not wanted/preferred" by the devs. I added documentation a few days ago to the test script and the INSTALL file for Horde 5.2.1 that indicates that using any 5.3 version is a bad idea.

07/20/2014 05:22:24 PM	torben (at) dannhauer (dot) info	Comment #5	Reply to this comment
What Michael R. says is correct. We can work around things when they are API related (i.e. PHP didn't support this feature until x.y.z). We are not going to bother to work around things, on an ancient version of PHP, when it is a bug. The way you fix bugs is to upgrade the program with the underlying issue ... in this case, PHP. Michael, thanks for the hint with the version, upgrading to 5.4 fixed my issue. However I agree, the test script should warn about an ancient PHP version which is not supported or "not wanted/preferred" by the devs. Thanks for your great work. Torben

07/10/2014 06:37:12 PM	Michael Slusarz	Comment #4	Reply to this comment
What Michael R. says is correct. We can work around things when they are API related (i.e. PHP didn't support this feature until x.y.z). We are not going to bother to work around things, on an ancient version of PHP, when it is a bug. The way you fix bugs is to upgrade the program with the underlying issue ... in this case, PHP.

07/10/2014 04:30:01 PM	Michael Rubinsky	Comment #3 Assigned to Michael Rubinsky State ⇒ Not A Bug	Reply to this comment
Correct. Update your ancient, buggy version of PHP.

07/10/2014 09:09:13 AM	arjen+horde (at) de-korte (dot) org	Comment #2	Reply to this comment
What would be the right way to deal with this? You have already confirmed that the problem is related to the PHP version, so upgrade PHP to a more recent version. If not you'll find yourself in a situation where you run into PHP bugs that have been fixed long ago, but still present in the version you're using. Version 5.3.3 was released 4 years ago, it is well past its use-by date now (and this is what you can expect people will tell you over-and-over again).

07/10/2014 08:51:56 AM	Jasper (dot) Olbrich (at) students (dot) uni-marburg (dot) de	Comment #1 Priority ⇒ 1. Low Type ⇒ Bug Summary ⇒ Smartmobile view fails to load with PHP 5.3.3 Queue ⇒ Horde Framework Packages Milestone ⇒ Patch ⇒ No State ⇒ Unconfirmed	Reply to this comment
After the update to Horde 5.2, the initial redirect from login.php to the initial application fails when using the smartmobile view. I end up in an infinite redirection loop. My observations so far: * the initial page is /imp/smartmobile.php#mailbox?mbox=SU5CT1g (correct) * the call to Horde::url() returns https://host:port/#mailbox?mbox=SU5CT1g (wrong) I assume the error is due to changes in parse_url(), because: on the production system (PHP Version 5.3.3-7+squeeze19) var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); array(1) { 'fragment' => string(20) "mailbox?mbox=SU5CT1g" } and with a more recent version (php -v: PHP 5.5.9-1ubuntu4 (cli) (built: Apr 9 2014 17:11:57)) var_dump(parse_url('/imp/smartmobile.php#mailbox?mbox=SU5CT1g')); array(2) { ["path"]=> string(20) "/imp/smartmobile.php" ["fragment"]=> string(20) "mailbox?mbox=SU5CT1g" } What would be the right way to deal with this?