Project

General

Profile

Actions
Copy link

Bug #20943

closed

Rudder incorectly parse URL with a '+' in the path into spaces

Added by François ARMAND over 2 years ago. Updated over 2 years ago.

Status:
Released
Priority:
N/A
Assignee:
Vincent MEMBRÉ
Category:
API
Target version:
6.2.14
Pull Request:
https://github.com/Normation/rudder/p...
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
Name check:
To do
Fix check:
Checked
Regression:

Description

It always transform it to a space character, but it should not do it for the PATH part, see https://stackoverflow.com/a/29948396 for an explanation.

I think it's only of importance for API that uses the path for resources, since we don't analyse path content by oursefves in other cases, only parameters (and these are OK AFAIK).

The correct correction would be to change lift decoding, but lift is ~ no more supported, and the sides effects may be extremelly far reaching (perhaps a ton of code in lift relies on that bug).

So I propose to only correct the problem in our API Path parsing, and at no other places.

Actions
Copy link

Also available in: Atom PDF