Skip to content

Alternative services support #5841

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.

Sign up for GitHub

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
Draft
vietj wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Alternative services support #5841

vietj wants to merge 3 commits into from
+1,128 −232

Conversation

@vietj
Copy link
Member

@vietj vietj commented Dec 9, 2025 •
edited
Loading

  • consolidate host / port parsing in a dedicated URI parsing class (it might be reused in uri-template project which has similar parsing rules) and ensure it conforms to ABNF
  • alt-svc header parser
  • ...

@vietj vietj added this to the 5.1.0 milestone Dec 9, 2025
@vietj vietj marked this pull request as draft December 9, 2025 23:47
@vietj
Move URI parsing code in an UriParser util class so this class contai…
98100b1 
…ns parsing code specifc to RFC3986 production rules.
@vietj vietj self-assigned this Dec 9, 2025
@vietj vietj linked an issue Dec 9, 2025 that may be closed by this pull request
Alternative services HTTP client support #5840
Open
vietj added 2 commits December 10, 2025 13:47
@vietj
HostAndPort IP-literal parsing should validate the in square brackett…
00a0321 
…ed content matches IPv6address or IPvFuture production rules.

Motivation:

The HostAndPort IP-literal parser attempts to find content between square brackets, it should ensure that the enclosed content matches production rule.

Changes:

Implement IPv6address and IPvFuture parsing and use it when parsing IP-literal.
@vietj
Implement Alt-Svc ABNF production parsing.
0914eb3 
Motivation:

Implement Alt-Svc ABNF production parsing into a proper AltSvc object representing the parsed string.

This is required to implement alternative services.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@vietj vietj

Labels

None yet

Projects

None yet

Milestone

5.1.0

Development

Successfully merging this pull request may close these issues.

Alternative services HTTP client support

2 participants

@vietj