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.

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

Alt-Svc Editorial Nits #78

Closed
wants to merge 1 commit into from
Closed

Alt-Svc Editorial Nits #78

wants to merge 1 commit into from

Conversation

MikeBishop
Copy link
Contributor

Assorted:

  • Alt-Svc doesn't introduce 421, just points to RFC 7540
  • Missing "the" in overview
  • Requests != connections in HTTP/2; presuming you mean connections throughout
  • Non-normative "can" mismatch with normative SHOULD later in the document
  • Clients who wish to avoid tracking are also permitted to just drop Alt-Used; turned to normative MAY to match earlier text
  • Word order in scheme confusion text

@MikeBishop
Alt-Svc Editorial Nits
6836690 
Assorted:
  - Alt-Svc doesn't introduce 421, just points to RFC 7540
  - Missing "the"
  - Requests != connections in HTTP/2; presuming you mean connections throughout
  - Non-normative "can" mismatch with normative SHOULD later in the document
  - Clients who wish to avoid tracking are also permitted to just drop Alt-Used; turned to normative MAY to match earlier text
  - Word order in scheme confusion text
reschke added a commit that referenced this pull request Aug 14, 2015
@reschke
Editorial improvements suggest by Mike Bishop in #78
65ffed0
@reschke
Copy link
Contributor

reschke commented Aug 14, 2015

(see code comments; cherry-picked everything else in 65ffed0)

@reschke reschke closed this Aug 14, 2015
@MikeBishop MikeBishop deleted the patch-2 branch May 10, 2021 19:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@MikeBishop @reschke