Project

General

Profile

Actions

Bug #20943

closed

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

Added by François ARMAND 8 months ago. Updated 6 months ago.

Status:
Released
Priority:
N/A
Category:
API
Target version:
Severity:
UX impact:
User visibility:
Effort required:
Priority:
0
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 #1

Updated by François ARMAND 8 months ago

  • Status changed from New to In progress
  • Assignee set to François ARMAND
Actions #2

Updated by François ARMAND 8 months ago

  • Status changed from In progress to Pending technical review
  • Assignee changed from François ARMAND to Vincent MEMBRÉ
  • Pull Request set to https://github.com/Normation/rudder/pull/4217
Actions #3

Updated by Vincent MEMBRÉ 8 months ago

  • Target version changed from 6.2.13 to 6.2.14
Actions #4

Updated by François ARMAND 7 months ago

  • Status changed from Pending technical review to Pending release
Actions #5

Updated by François ARMAND 6 months ago

It's OK but the parsing of id is not correct, making that change a bit useless

Actions #7

Updated by Vincent MEMBRÉ 6 months ago

  • Status changed from Pending release to Released

This bug has been fixed in Rudder 6.2.14 which was released today.

Actions

Also available in: Atom PDF