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

Add handler for HTTP/2 PRI #159

Open
glaslos opened this issue May 7, 2024 · 2 comments · May be fixed by #178
Open

Add handler for HTTP/2 PRI #159

glaslos opened this issue May 7, 2024 · 2 comments · May be fixed by #178
Labels
protocol An issue for a new protocol

Comments

@glaslos
Copy link
Member

glaslos commented May 7, 2024

See for context: RFC

@glaslos glaslos added the protocol An issue for a new protocol label Dec 27, 2024
@Beeram12
Copy link

Beeram12 commented Feb 19, 2025

Hii @glaslos
I just wanted to ask regarding the flow to solve this issue

  1. I have to write the spicy grammer(parser implementation) for HTTP/2
  2. Implementing HTTP/2 PRI handler
  3. Integrating with exisiting code
  4. Update makefile
  5. Write tests

Is this the correct flow to solve this issue.If you can add anything that would be very much helpful

@glaslos
Copy link
Member Author

glaslos commented Feb 23, 2025

@Beeram12 This issue is an adjustment of existing code, not yet utilizing Spicy for protocol parsing.
Your entry point is here consider what you need to change here to handle the HTTP/2 PRI request. Then modify the HTTP handler to reply to the request correctly.

@Beeram12 Beeram12 linked a pull request Mar 8, 2025 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
protocol An issue for a new protocol
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants