1273225 - Certain characters in URL no longer working with latest Firefox version

Status: RESOLVED FIXED

(Web Compatibility :: Site Reports, defect)

Description

[Jordan]

User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_11_3) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/49.0.2623.112 Safari/537.36

Steps to reproduce:

Certain characters that used to work in Firefox in URL's no longer are recognized. This URL used to work:

https://marketbolt.com/system/members/panel.php?a=1#Custom-Page;{%27template%27:%27custom/jeunesse%27}

(Note: This URL is a members only URL and you must be logged in to use it)

The %27 is actually a ' quotation mark but gets changed when I post the link here. Previous versions of FireFox (before 45 I believe) used to recognized this URL. Not it does not. My marketing system software relies on this URL structure. It works in Chrome, Safari, and Explorer.


Actual results:

Instead of https://marketbolt.com/system/members/panel.php?a=1#Custom-Page;{%27template%27:%27custom/jeunesse%27}

Firefox drops everything within the brackets leaving this:

https://marketbolt.com/system/members/panel.php?a=1#Custom-Page;{}


Expected results:

Firefox should leave the URL intact like this:

https://marketbolt.com/system/members/panel.php?a=1#Custom-Page;{%27template%27:%27custom/jeunesse%27}

Comment 1

[Jordan]

You'll need to login to be able to test the URL. Here is a demo login:

https://marketbolt.com/login

email: [e-mail]
password: abc123

Comment 2

[Loic]

Short URL for the link: https://goo.gl/zmTaoq

Comment 3

[Loic]

I guess you were using FF38 ESR before 45.

regression range:
https://hg.mozilla.org/mozilla-central/pushloghtml?fromchange=3d846527576f&tochange=63006936ab99

regressed by bug 1093611.

Probably a dupe of bug 1213870.

Comment 4

[Valentin Gosu [:valentin] (he/him)]

I don't think this is an actual Firefox regression.
For example, you can load https://example.org/system/members/panel.php?a=1#Custom-Page;{%27template%27:%27custom/jeunesse%27} and it works.
The code at marketbolt.com will probably need changing.

Comment 5

[Honza Bambas (:mayhemer)]

Changing component based on comment 4.

Comment 6

[Mike Taylor [:miketaylr]]

Hi Jordan,

Can you explain how your app is handling the URLs? Like Comment #4 shows, https://example.org/system/members/panel.php?a=1#Custom-Page;{%27template%27:%27custom/jeunesse%27} can be loaded and the contents inside {} aren't dropped.

Comment 7

[Jordan]

I have a developer looking at the software (affiliate software) and is unable to find the bug. It was working about 4-5 months ago and then stopped working after a certain Firefox release. I'm not sure which release caused this to happen but it was probably build 45.

Comment 8

[Valentin Gosu [:valentin] (he/him)]

Bug 1093611 changed the handling of .hash - it would now return a percent encoded string.
I suggest looking at all the JS code that uses location.hash - replacing that with decodeURIComponent(location.hash) should work.

Comment 9

[Jordan]

Thank you Valentin! That solved it!

Comment 10

[Valentin Gosu [:valentin] (he/him)]

(In reply to Jordan from comment #9)
> Thank you Valentin! That solved it!

Glad to hear that!