Skip to main content

Bug Tracker

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 rwaldron comment:1

milestone: 1.4.11.5
owner: → rwaldron
priority: minorlow
status: newassigned
type: bugenhancement

Changed December 27, 2010 10:37PM UTC by rwaldron comment:2

keywords: → ajaxrewrite

Changed January 09, 2011 08:50PM UTC by jaubourg comment:3

resolution: → fixed
status: assignedclosed

Fixes #5812. =? will be detected even when it has been escaped during data serialization.

Changeset: 8c8bd3bf6ac982f98c1b89b0af74a1fd2d07e360

Changed January 10, 2011 04:36PM UTC by jaubourg comment:4

#4920 is a duplicate of this ticket.

Changed January 25, 2011 11:57PM UTC by jaubourg comment:5

resolution: fixed
status: closedreopened

Changed January 25, 2011 11:58PM UTC by jitter comment:6

_comment0: This enhancement breaks legitimate uses of ? in url or data.1296176321563314
blockedby: → 8054
resolution: → wontfix
status: reopenedclosed

This enhancement breaks legitimate uses of ? in url or data. The fix was reverted here 325dcdc