• B
    http1connection: Make content-length parsing more strict · bf90f3a9
    Ben Darnell 提交于
    Content-length and chunk size parsing now strictly matches the RFCs.
    We previously used the python int() function which accepted leading
    plus signs and internal underscores, which are not allowed by the
    HTTP RFCs (it also accepts minus signs, but these are less problematic
    in this context since they'd result in errors elsewhere)
    
    It is important to fix this because when combined with certain proxies,
    the lax parsing could result in a request smuggling vulnerability (if
    both Tornado and the proxy accepted an invalid content-length but
    interpreted it differently). This is known to occur with old versions
    of haproxy, although the current version of haproxy is unaffected.
    bf90f3a9
httpserver_test.py 49.1 KB