aio-libs / aio-libs/aiohttp

llhttp should be a separate, optional package

オープン
#8,393 コメント 2 件 リアクション 1 件 担当者 0 名 GitHub で見る
enhancement
主要言語
Python
スター
16.5k
フォーク
2.4k
平均マージ
17時間 22分
マージ済み PR(30日)
212

説明

### Is your feature request related to a problem?

Currently there are several projects having python bindings for `llhttp`. There is
- https://github.com/pallas/pyllhttp
- https://github.com/geventhttpclient/geventhttpclient
- aiohttp

And I probably missed a couple of other ones. `pyllhttp` hasn't been updated for a long time and `geventhttpclient` also struggles with maintenance. Having a single, well maintained bindings package would profit the python ecosystem and share the maintenance in more hands.

At the same time, there are requests from people using alternative python implementations asking for pure python packages for easier integration:
- https://github.com/aio-libs/aiohttp/issues/7632
- https://github.com/aio-libs/aiohttp/issues/6584
- https://github.com/aio-libs/aiohttp/issues/7802

Separating llhttp into a separate package would make their life easier as well.

### Describe the solution you'd like

`aiohttp` features the best maintained python bindings for `llhttp`. It would be great, if not only `aiohttp` could profit from them. Separating the `llhttp` bindings into their own package and having it available as a readily available HTTP building block in line with other packages like `multidict`, `h11` etc. would profit the python HTTP eco system as a whole and also make life easier for `aiohttp` users of non-standard python implementations.

### Describe alternatives you've considered

`pyllhttp` and `geventhttpclient` have their own bindings using a plain C extension. Though, the cython code of `aiohttp` is imho nicer to read for the python user and less error prone than the [c code within geventhttpclient](https://github.com/geventhttpclient/geventhttpclient/blob/master/ext/_parser.c). As all bindings just wrap around `llhttp`, I guess the performance differences should be marginal.

### Additional context

This is more like suggesting a restructuring and not a new feature. So I'm not that sure if it really belongs here or better into the discussion section. Please just move it there, if you think it fits better.

### Code of Conduct

- [X] I agree to follow the aio-libs Code of Conduct

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。