Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

port hyperium/* as rama-http-core #347

Merged
merged 48 commits into from
Dec 28, 2024
Merged

port hyperium/* as rama-http-core #347

merged 48 commits into from
Dec 28, 2024

Conversation

GlenDC
Copy link
Member

@GlenDC GlenDC commented Nov 3, 2024

  • dropped logic we do not need
  • dropped wrapper/duplicate code no longer needed
  • modify signatures to avoid having to need to lock
  • keep file structure as similar as possible to easier facilitate keeping in sync with upstream (already proven to work as this PR already contains a couple of syncs with upstream, was very easy to do)

Closes #86

next steps actually migrate and port the logic
@GlenDC GlenDC self-assigned this Nov 3, 2024
@GlenDC GlenDC changed the title port hyper and hyper-util as rama-http-core port hyperium/* as rama-http-core Nov 3, 2024
for encoding one can still opt-in for title-casing if ever needed
Changelog:

## What's Changed

- Fix treating HEADERS frames with a non-zero content-length but END_STREAM flag as malformed.
- Fix notifying the stream task when automatically reset on receipt of a stream error.

Release url: <https://github.com/hyperium/h2/releases/tag/v0.4.7>
raw changelog: <hyperium/h2@v0.4.6...v0.4.7>
Changelog:

### v1.5.1 (2024-11-19)

#### Bug Fixes

* **http2:**
  * pass proper value to h2 max_local_error_reset_streams ([4a20147a](hyperium/hyper@4a20147))
  * improve graceful shutdown during handshake (#3729) ([13b05943](hyperium/hyper@13b0594))

Release url: <https://github.com/hyperium/hyper/releases/tag/v1.5.1>
Raw changelog: <hyperium/hyper@v1.5.0...v1.5.1>
Changelog:

### v1.5.2 (2024-12-16)

#### Bug Fixes

* **http1:**
  * fix intermitent panic parsing partial headers (#3812) ([a131111f](hyperium/hyper@a131111), closes [#3811](hyperium/hyper#3811))
  * skip debug assertion of content length for HEAD responses (#3795) ([eaf2267c](hyperium/hyper@eaf2267), closes [#3794](hyperium/hyper#3794))

Release url: <https://github.com/hyperium/hyper/releases/tag/v1.5.2>
Raw changelog: <hyperium/hyper@v1.5.1...v1.5.2>
@GlenDC GlenDC marked this pull request as ready for review December 28, 2024 20:47
Copy link
Member Author

@GlenDC GlenDC left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

GTG

@GlenDC GlenDC enabled auto-merge (squash) December 28, 2024 21:04
@GlenDC GlenDC disabled auto-merge December 28, 2024 21:20
@GlenDC GlenDC merged commit 360fb80 into main Dec 28, 2024
34 checks passed
@GlenDC GlenDC deleted the feat/rama-http-core branch December 28, 2024 21:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Embed hyper and friends directly into rama
1 participant