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

initial editorial comments #2446

Closed
m4mb01t4l14n0 opened this issue Jul 9, 2022 · 1 comment
Closed

initial editorial comments #2446

m4mb01t4l14n0 opened this issue Jul 9, 2022 · 1 comment

Comments

@m4mb01t4l14n0
Copy link

Abstract: ... there is no existing way for the origin to share such a nonce without exposing the fact that they serve resources ...

s/they serve/it serves

Also, some might consider this a run-on sentence. An abstract's brevity might condone such behavior, of course.

Section 1: Here I think it's better to have shorter sentences. I support directly stating that non-probability is a key feature, but I feel that requires an explanation. I don't have better wording yet - will consider and propose later. I find the second paragraph a bit abrupt having just stated the reasons non-probability is useful. There needs to be a caveat here, thus: "There are scenarios where servers may want to expose that authentication is required for access to specific resources. This is left for future work."

Section 5: ...HTTP intermediaries that support this specification will validate the authentication received from the client themselves, then let the upstream HTTP server using some other mechanism....

incomplete sentence. Likely "...let the upstream HTTP server AUTHENTICATE THE CLIENT using some other mechanism."

@DavidSchinazi
Copy link
Contributor

Thanks! Fixed via 51404e94ab0180e6d15556af7d20a1e30388a6db

@mnot mnot transferred this issue from DavidSchinazi/draft-schinazi-httpbis-transport-auth Feb 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants