aio-libs / aio-libs/aiohttp

llhttp should be a separate, optional package

未关闭
#8,393 2 条评论 1 个 reaction 已指派 0 人 在 GitHub 查看
enhancement
主要语言
Python
星标
16.5k
派生
2.4k
平均合并
17 小时 22 分钟
30 天内合并 PR
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 摘要。