Edgewall Software

Ticket #2042 (closed defect: fixed)

Opened 3 years ago

Last modified 3 years ago

& characters in query: links are escaped twice

Reported by: mgood Owned by: cboos
Priority: high Milestone: 0.9
Component: general Version: 0.9b1
Severity: normal Keywords:
Cc:

Description

The & characters in query: links (possibly others, but I haven't tested this yet) are escaped twice to: &

Attachments

2042-fix.patch (3.6 kB) - added by cboos 3 years ago.
Fixes query: and search: links. Takes into account #1927.

Change History

Changed 3 years ago by cboos

  • owner changed from mgood to cboos
  • status changed from new to assigned
  • version changed from 0.8.4 to 0.9b1

I noticed that too while working on the search: links, and I fixed it for the search, but not for the query:

Changed 3 years ago by cboos

Oops, sorry, just looked closer to ticket #1927 and learnt that ampersands in URIs should be escaped...

I'll replace the patch soon.

Changed 3 years ago by cboos

Fixes query: and search: links. Takes into account #1927.

Changed 3 years ago by cboos

  • priority changed from normal to high

That patch also fixes the query: links which start without "?". Before, a Query.from_string was done on an already escaped string, therefore the split on "&" produced an incorrect field name (e.g. query:milestone=1.0&owner=me was transformed in amp;owner=me&milestone=1.0).

Changed 3 years ago by cmlenz

cboos, I've tested this patch and it looks good. Can you please check it in?

Changed 3 years ago by cboos

  • status changed from assigned to closed
  • resolution set to fixed

Patch applied in r2273.

Add/Change #2042 (& characters in query: links are escaped twice)

Author



Change Properties
<Author field>
Action
as closed
Next status will be 'reopened'
 
Note: See TracTickets for help on using tickets.