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

Send status 100 "Continue" #36

Open
Luukdegram opened this issue Jun 14, 2021 · 1 comment
Open

Send status 100 "Continue" #36

Luukdegram opened this issue Jun 14, 2021 · 1 comment
Labels
enhancement New feature or request good first issue Good for newcomers
Milestone

Comments

@Luukdegram
Copy link
Owner

When a client requests with a header Expect: 100-continue, we must send a status 100 continue. We can send this immediately after verifying the request headers and MUST send this before sending the 'regular' response.

As we're currently relying on buffered writers for performance benefits, the 100 status code must be flushed immediately to the client, so they can send their body (if they haven't already).

@Luukdegram Luukdegram added enhancement New feature or request good first issue Good for newcomers labels Jun 14, 2021
@Luukdegram Luukdegram added this to the HTTP 1.1 milestone Jun 14, 2021
@Avokadoen
Copy link
Contributor

Avokadoen commented Jun 24, 2021

Relevant specification:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants