File 1521_delegate_parseqs_stdlib_bpo42967.patch of Package python-Twisted

Overview Repositories Revisions Requests Users Attributes Meta

File 1521_delegate_parseqs_stdlib_bpo42967.patch of Package python-Twisted (Revision 54353248586ae9528399d5bda5dde341)

Currently displaying revision 54353248586ae9528399d5bda5dde341 , Show latest

From df79d69adea5c819bb104861dccf1bbe25851644 Mon Sep 17 00:00:00 2001
From: Thomas Grainger <tagrain@gmail.com>
Date: Sun, 21 Feb 2021 11:54:25 +0000
Subject: [PATCH 1/2] delegate to stdlib parse qs

---
 src/twisted/web/http.py                    |   26 +-------------------------
 src/twisted/web/newsfragments/10096.bugfix |    1 +
 src/twisted/web/server.py                  |    5 ++---
 3 files changed, 4 insertions(+), 28 deletions(-)
 create mode 100644 src/twisted/web/newsfragments/10096.bugfix

--- a/src/twisted/web/http.py
+++ b/src/twisted/web/http.py
@@ -112,6 +112,7 @@ from urllib.parse import (
     ParseResultBytes,
     urlparse as _urlparse,
     unquote_to_bytes as unquote,
+    parse_qs,
 )
 from typing import Callable
 
@@ -263,31 +264,6 @@ def urlparse(url):
     return ParseResultBytes(scheme, netloc, path, params, query, fragment)
 
 
-def parse_qs(qs, keep_blank_values=0, strict_parsing=0):
-    """
-    Like C{cgi.parse_qs}, but with support for parsing byte strings on Python 3.
-
-    @type qs: C{bytes}
-    """
-    d = {}
-    items = [s2 for s1 in qs.split(b"&") for s2 in s1.split(b";")]
-    for item in items:
-        try:
-            k, v = item.split(b"=", 1)
-        except ValueError:
-            if strict_parsing:
-                raise
-            continue
-        if v or keep_blank_values:
-            k = unquote(k.replace(b"+", b" "))
-            v = unquote(v.replace(b"+", b" "))
-            if k in d:
-                d[k].append(v)
-            else:
-                d[k] = [v]
-    return d
-
-
 def datetimeToString(msSinceEpoch=None):
     """
     Convert seconds since epoch to HTTP datetime string.
--- /dev/null
+++ b/src/twisted/web/newsfragments/10096.bugfix
@@ -0,0 +1 @@
+delegate to urllib.parse:parse_qs in twisted.web.http:parse_qs to avoid CVE-2021-23336 and the associated CI failures
--- a/src/twisted/web/server.py
+++ b/src/twisted/web/server.py
@@ -19,7 +19,7 @@ import os
 import re
 from html import escape
 from typing import List, Optional
-from urllib.parse import quote as _quote
+from urllib.parse import quote as _quote, unquote_to_bytes as _unquote_to_bytes
 
 import zlib
 from binascii import hexlify
@@ -31,7 +31,6 @@ from twisted.spread.pb import Copyable,
 from twisted.internet import address, interfaces
 from twisted.internet.error import AlreadyCalled, AlreadyCancelled
 from twisted.web import iweb, http, util
-from twisted.web.http import unquote
 from twisted.python import reflect, failure, components
 from twisted import copyright
 from twisted.web import resource
@@ -213,7 +212,7 @@ class Request(Copyable, http.Request, co
 
         # Resource Identification
         self.prepath = []
-        self.postpath = list(map(unquote, self.path[1:].split(b"/")))
+        self.postpath = [_unquote_to_bytes(v) for v in self.path[1:].split(b"/")]
 
         # Short-circuit for requests whose path is '*'.
         if self.path == b"*":

Places

File 1521_delegate_parseqs_stdlib_bpo42967.patch of Package python-Twisted (Revision 54353248586ae9528399d5bda5dde341)

Places