GHSA-m5ff-3wj3-8ph4

Source

https://github.com/advisories/GHSA-m5ff-3wj3-8ph4

Import Source

https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2019/12/GHSA-m5ff-3wj3-8ph4/GHSA-m5ff-3wj3-8ph4.json

JSON Data

https://api.test.osv.dev/v1/vulns/GHSA-m5ff-3wj3-8ph4

Published

2019-12-26T16:34:38Z

Modified

2025-02-17T05:30:11.337257Z

Summary

HTTP Request Smuggling: Invalid whitespace characters in headers in Waitress

Details

Impact

If a proxy server is used in front of waitress, an invalid request may be sent by an attacker that bypasses the front-end and is parsed differently by waitress leading to a potential for HTTP request smuggling.

Content-Length: 10
Transfer-Encoding: [\x0b]chunked

For clarity:

0x0b == vertical tab

Would get parsed by Waitress as being a chunked request, but a front-end server would use the Content-Length instead as the Transfer-Encoding header is considered invalid due to containing invalid characters.

If a front-end server does HTTP pipelining to a backend Waitress server this could lead to HTTP request splitting which may lead to potential cache poisoning or unexpected information disclosure.

Patches

Please upgrade to Waitress 1.4.1 which fixes this issue with stricter HTTP field validation.

Waitress 1.4.1 due to this change has become much more strict in what is allowed in header values, while the maintainers don't believe that these changes will cause any issues, it may cause failures with non-conformist reverse proxies or clients, and it is highly recommend that users validate the changes in their environment and make sure it won't cause any unacceptable failures.

Workarounds

You may enable additional protections on front-end servers, those that follow RFC7230 correctly would drop the request with a 400 Bad Request.

Waitress will now correctly responds to the request with a 400 Bad Request, and will drop the connection to avoid any potential HTTP pipelining issues.

References

This was mentioned in https://portswigger.net/research/http-desync-attacks-what-happened-next and was specifically mentioned as being an issue in HAProxy which did not properly filter it in this article: https://nathandavison.com/blog/haproxy-http-request-smuggling

Thanks

The Pylons Project would like to thank ZeddYu Lu for doing extended testing against Waitress 1.4.0 and bringing this to our attention!

For more information

If you have any questions or comments about this advisory:

open an issue at https://github.com/Pylons/waitress/issues (if not sensitive or security related)
email the Pylons Security mailing list: pylons-project-security@googlegroups.com (if security related)

Database specific

{
    "nvd_published_at": null,
    "cwe_ids": [
        "CWE-444"
    ],
    "severity": "HIGH",
    "github_reviewed": true,
    "github_reviewed_at": "2019-12-26T16:34:06Z"
}

References

Affected packages

PyPI / waitress

Package

Name: waitress; View open source insights on deps.dev
Purl: pkg:pypi/waitress

Affected ranges

Type: ECOSYSTEM
Events: Introduced

0Unknown introduced version / All previous versions are affected

Fixed

1.4.1

Affected versions

0.*

0.1

0.2

0.3

0.4

0.5

0.6

0.6.1

0.7

0.8

0.8.1

0.8.2

0.8.3

0.8.4

0.8.5

0.8.6

0.8.7

0.8.8

0.8.9

0.8.10

0.8.11b0

0.9.0b0

0.9.0b1

0.9.0

1.*

1.0a1

1.0a2

1.0.0

1.0.1

1.0.2

1.1.0

1.2.0b1

1.2.0b2

1.2.0b3

1.2.0

1.2.1

1.3.0b0

1.3.0

1.3.1

1.4.0