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

import 421 (Misdirected Request) Status Code from h2 #622

Closed
royfielding opened this issue Dec 31, 2020 · 6 comments · Fixed by #624 or #627
Closed

import 421 (Misdirected Request) Status Code from h2 #622

royfielding opened this issue Dec 31, 2020 · 6 comments · Fixed by #624 or #627
Assignees

Comments

@royfielding
Copy link
Member

I think we need to copy the 421 status code definition from Section 9.1.2 of RFC7540 since it is generally applicable for rejecting a misdirected request.

@royfielding royfielding self-assigned this Dec 31, 2020
@royfielding
Copy link
Member Author

curious if @martinthomson has an opinion on moving 421 to http-core

@martinthomson
Copy link
Contributor

A slight preference for moving it, I think.

@mnot
Copy link
Member

mnot commented Jan 2, 2021

Since we're doing h2bis and since it is applicable more broadly, I agree.

@reschke
Copy link
Contributor

reschke commented Jan 2, 2021

When doing so, we need to consider https://www.rfc-editor.org/errata/eid4666

@royfielding
Copy link
Member Author

Hmm, the errata is about the cacheable paragraph (which is now heuristically) and I was going to remove that for the same reason but decided to import without changes first. I am just going to remove it now.

@reschke
Copy link
Contributor

reschke commented Jan 3, 2021

That might work as well; but then we'll have to mention in "changes" that the default changed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment