Skip to content

Commit db5312b

Browse files
committed
fix(web): issue2551381 - roundup-server parses multiple '?" incorrectly.
Change 'rfind()' to 'find()' to locate the first unencoded/unescaped '?' in a URL. End the path component of the url at that point. This matches https://www.rfc-editor.org/rfc/rfc3986#section-3.4.
1 parent 7c9247d commit db5312b

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

roundup/scripts/roundup_server.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -398,7 +398,7 @@ def inner_run_cgi(self):
398398

399399
return
400400

401-
i = self.path.rfind('?')
401+
i = self.path.find('?')
402402
if i >= 0:
403403
# rest starts with /, query is without ?
404404
rest, query = self.path[:i], self.path[i+1:]

0 commit comments

Comments
 (0)