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

Jump to bottom

Moving HTTP/2 stream vars into local context #10877

Closed
wants to merge 1 commit into from
Closed

Moving HTTP/2 stream vars into local context #10877

wants to merge 1 commit into from

Conversation

icing
Copy link
Contributor

@icing icing commented Mar 31, 2023

  • remove NGHTTP2 members of struct HTTP
  • add void *impl_ctx to struct HTTP
  • manage local stream_ctx in http2.c

@github-actions github-actions bot added the HTTP label Mar 31, 2023
sonatype-lift[bot]
sonatype-lift bot reviewed Mar 31, 2023
View reviewed changes
lib/http2.c Outdated Show resolved Hide resolved
@icing
Moving HTTP/2 stream vars into local context
02c8c12 
- remove NGHTTP2 members of `struct HTTP`
- add `void *h2_ctx` to `struct HTTP`
- add `void *h3_ctx` to `struct HTTP`
- separate h2/h3 pointers are needed for eyeballing
- manage local stream_ctx in http implementations
@bagder bagder closed this in 632e0fb Apr 6, 2023
bch pushed a commit to bch/curl that referenced this pull request Jul 19, 2023
@icing @bch
http2: move HTTP/2 stream vars into local context
4850c3e 
- remove NGHTTP2 members of `struct HTTP`
- add `void *h2_ctx` to `struct HTTP`
- add `void *h3_ctx` to `struct HTTP`
- separate h2/h3 pointers are needed for eyeballing
- manage local stream_ctx in http implementations

Closes curl#10877
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sonatype-lift sonatype-lift[bot] sonatype-lift left review comments

Assignees
No one assigned
Labels
HTTP
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@icing