magnusmanske / petscan_rs

The repo for the PetScan tool
https://petscan.wmflabs.org/
GNU General Public License v3.0
43 stars 10 forks source link

Fix #132 by adjusting querystring processing #140

Closed mwootten closed 5 months ago

mwootten commented 10 months ago

The backend decodes querystrings with the qstring crate. However, that decodes both a plus sign and a URL-encoded plus sign to exactly the same thing, erasing the distinction between plus signs and spaces (see algesten/qstring#3). This patch instead uses the underlying url crate, which handles plus signs correctly.

This fix should be combined with #138, which fixes the frontend piece; however, merging just this one would correct the saving in the backend.