Side navigation
#5812 closed enhancement (wontfix)
Opened January 15, 2010 05:47AM UTC
Closed January 25, 2011 11:58PM UTC
Should look for =%3F in addition to =?
Reported by: | harrisms | Owned by: | rwaldron |
---|---|---|---|
Priority: | low | Milestone: | 1.5 |
Component: | ajax | Version: | 1.4 |
Keywords: | ajaxrewrite | Cc: | |
Blocked by: | Blocking: |
Description
The ajax function searches for =? in the data string to replace it with =jsonp..., but it does that *after* the data string is constructed by the param function, which uses encodeURIComponent to turn =? into =%F.
A quick fix is to modify the jsre regex to match =%3F too.
Attachments (0)
Change History (6)
Changed October 22, 2010 10:44PM UTC by comment:1
milestone: | 1.4.1 → 1.5 |
---|---|
owner: | → rwaldron |
priority: | minor → low |
status: | new → assigned |
type: | bug → enhancement |
Changed December 27, 2010 10:37PM UTC by comment:2
keywords: | → ajaxrewrite |
---|
Changed January 09, 2011 08:50PM UTC by comment:3
resolution: | → fixed |
---|---|
status: | assigned → closed |
Fixes #5812. =? will be detected even when it has been escaped during data serialization.
Changeset: 8c8bd3bf6ac982f98c1b89b0af74a1fd2d07e360
Changed January 25, 2011 11:57PM UTC by comment:5
resolution: | fixed |
---|---|
status: | closed → reopened |
Changed January 25, 2011 11:58PM UTC by comment:6
_comment0: | This enhancement breaks legitimate uses of ? in url or data. → 1296176321563314 |
---|---|
blockedby: | → 8054 |
resolution: | → wontfix |
status: | reopened → closed |
This enhancement breaks legitimate uses of ? in url or data. The fix was reverted here 325dcdc