Version 0.19.0

[Kludex]

Checklist

• Remove --debug flag #1640
• Revert "Use updated headers from server state" #1671
• Upgrade request handling: ignore http/2 and optionally ignore websocket #1661
• Bump minimal httptools version to 0.5.0 #1645
• Support Python 3.11 #1652

Diff: 0.18.3...master
Draft: https://github.com/encode/uvicorn/releases/tag/untagged-8b346412076c82a9e8cc

[nlsj1985]

@Kludex httptools 5.0.0 just got released with the CVE fixes in llhttp. It seems it will get included in uvicorn automatically when you create a release, but i'm not sure. Perhaps, it's best to bump the version explicitly... thanks for checking it.

[Kludex]

@Kludex httptools 5.0.0 just got released with the CVE fixes in llhttp. It seems it will get included in uvicorn automatically when you create a release, but i'm not sure. Perhaps, it's best to bump the version explicitly... thanks for checking it.

• Bump minimal httptools version to 0.5.0 #1645 👍

[Kludex]

I'd like to make a release soon. 🙏

[nlsj1985]

note. I made a PR in the httptools project for the latest llhttp (v6.0.10) update.. but guess it's wise to first proceed with this httptools 0.5.0 in uvicorn 0.19.0 and when we get a new httptools version, queue it for the following uvicorn...

[Kludex]

Would you mind sharing the link?

[Kludex]

note. I made a PR in the httptools project for the latest llhttp (v6.0.10) update.. but guess it's wise to first proceed with this httptools 0.5.0 in uvicorn 0.19.0 and when we get a new httptools version, queue it for the following uvicorn...

But there's nothing uvicorn needs to do... Unless the bump solved any CVE.

[nlsj1985]

httptools v5.0.0 contains llhttp release/v6.0.9, this included some fixes for the 3 CVE's that where done in llhttp release/v6.0.7

llhttp release/v6.0.10 seems to update a resolution for CVE-2022-32213:
Disable chunked on obs by @ShogunPanda in nodejs/llhttp#196

nodejs https://github.com/nodejs/node/releases/tag/v18.9.1 has a paragraph about llhttp that mentions a bit more:
bypass via obs-fold mechanic (Medium)(CVE-2022-32213 ): The llhttp parser in the http module does not correctly parse and validate Transfer-Encoding headers. This can lead to HTTP Request Smuggling (HRS).

There are 3 change's in src/llhttp/http.ts:
nodejs/llhttp@f49fbf5

So without reverse engineering the entire llhttp release/v6.0.10 commit it seemed to me there's actual improvements made for the CVE in relation to obs-fold's .. that i guess where missed in the patch for the same CVE. If you want to know more, please check the patch or ask ShokunPanda who makes the patches for the llhttp project.

I'm just a random user that's trying to be instrumental to get fixes into uvicorn asap.

[Kludex]

This is the link I wanted: MagicStack/httptools#86 🙏

I'm just a random user that's trying to be instrumental to get fixes into uvicorn asap.

uvicorn doesn't pin httptools. I've only bumped the minimum requirement of httptools to 0.5.0 to force users to bump it, but it was actually not needed. We are only blocked by httptools here.

[ShogunPanda]

@nlsj1985 I can confirm folks. All OBS fold CVEs are fixed in 6.0.10

Next week I plan to release llhttp 7 with more features about passable callbacks and chunked encoding parsing, just FYI.

[Kludex]

@nlsj1985 I can confirm folks. All OBS fold CVEs are fixed in 6.0.10

Next week I plan to release llhttp 7 with more features about passable callbacks and chunked encoding parsing, just FYI.

Thanks for coming here, and let us know. Much appreciated! 🙏

[ShogunPanda]

@nlsj1985 I can confirm folks. All OBS fold CVEs are fixed in 6.0.10

Next week I plan to release llhttp 7 with more features about passable callbacks and chunked encoding parsing, just FYI.

Thanks for coming here, and let us know. Much appreciated! 🙏

You're welcome! 😉

[Kludex]

• Only Use updated headers from server state #1706 missing for 0.19.0 to be released.

[Kludex]

Ready for release. GitHub release draft ready as well.

[tomchristie]

Great. 🌟

[Kludex]

I'll release it tonight. Thanks @tomchristie 🙏