Bug Tracker

Opened 7 years ago

Closed 6 years ago

#5812 closed enhancement (wontfix)

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: #8054 Blocking:


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.

Change History (6)

comment:1 Changed 6 years ago by rwaldron

  • Milestone changed from 1.4.1 to 1.5
  • Owner set to rwaldron
  • Priority changed from minor to low
  • Status changed from new to assigned
  • Type changed from bug to enhancement

comment:2 Changed 6 years ago by rwaldron

  • Keywords ajaxrewrite added

comment:3 Changed 6 years ago by jaubourg

  • Resolution set to fixed
  • Status changed from assigned to closed

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

Changeset: 8c8bd3bf6ac982f98c1b89b0af74a1fd2d07e360

comment:4 Changed 6 years ago by jaubourg

#4920 is a duplicate of this ticket.

comment:5 Changed 6 years ago by jaubourg

  • Resolution fixed deleted
  • Status changed from closed to reopened

comment:6 Changed 6 years ago by jaubourg

  • Blocked by 8054 added
  • Resolution set to wontfix
  • Status changed from reopened to closed

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

Last edited 6 years ago by jitter (previous) (diff)
Note: See TracTickets for help on using tickets.