data.decodeUrl not working with cookie strings
Reported by Stephen Elson | July 17th, 2009 @ 01:32 PM | in 1.6
"BBC-UID=2438821bbf282c86b56114e7d04051202db7190ee028201082b09ce4ac07ad3d0libwww%2dperl%2f5%2e808; BBCNewsAudience=Domestic; hp=+acv+ba+e+c1+f1+da-i+g12; BBCNewsAudcWght=-99; BBC-UID=c42800e8fa2c490fa1ec14aed1a8bf4938d227a330c0b141c29233b3f454c5e50Mozilla%2f5%2e0%20%28Windows%3b%20U%3b%20Windows%20NT%205%2e1%3b%20en%2dGB%3b%20rv%3a1%2e8%2e1%2e14%29%20Gecko%2f20080404%20Firefox%2f2%2e0%2e0%2e14"
Seems to die silently on that.
Comments and changes to this ticket
-
Stephen Elson July 17th, 2009 @ 01:33 PM
This is because of the semi-colon separators, decodeUrl only handles ampersand separators. Perhaps a quick fix would be to replace ; with & before processing? Is this safe?
-
Stephen Elson July 17th, 2009 @ 07:25 PM
- Tag changed from “defect” to “data, defect”
-
Stephen Elson July 24th, 2009 @ 02:28 PM
- State changed from “new” to “accepted”
-
Jake Archibald August 3rd, 2009 @ 04:28 PM
- Assigned user set to “Jake Archibald”
- State changed from “accepted” to “open”
One more quick'un
-
Stephen Elson August 3rd, 2009 @ 05:00 PM
- State changed from “open” to “review”
(from [635e0e2b81f83f3b9e087ff48cb730b912daeb91]) glow.data.decodeUrl: Fixing for ; separator [#16 state:review] http://github.com/glow/glow1/commit/635e0e2b81f83f3b9e087ff48cb730b...
-
Jake Archibald August 5th, 2009 @ 02:04 PM
- State changed from “review” to “resolved”
Please Sign in or create a free account to add a new ticket.
With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.
Create new ticket
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile ยป
A JavaScript Library
People watching this ticket
Jake Archibald
Stephen Elson
Referenced by
-
16
data.decodeUrl not working with cookie strings
(from [635e0e2b81f83f3b9e087ff48cb730b912daeb91])
glow.da...