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

Alias proxy status, Erik Kline review #2667

Closed
tfpauly opened this issue Oct 25, 2023 · 3 comments · Fixed by #2668
Closed

Alias proxy status, Erik Kline review #2667

tfpauly opened this issue Oct 25, 2023 · 3 comments · Fixed by #2668

Comments

@tfpauly
Copy link
Contributor

tfpauly commented Oct 25, 2023

Internet AD comments for draft-ietf-httpbis-alias-proxy-status-05

CC @ekline

  • comment syntax:
  • "Handling Ballot Positions":

Comments

S2

  • Why is the value defined as a String that can be comma-separated as opposed
    to a List (of Strings)?

S3

  • Up to you, but feel free to reference RFC 3493 section 6.1 for getaddrinfo
    and discussion of the AI_CANONNAME flag.

Nits

S3

  • "might not available" ->
    "might not be available"
@tfpauly
Copy link
Contributor Author

tfpauly commented Oct 25, 2023

The nit was fixed with #2647

@tfpauly
Copy link
Contributor Author

tfpauly commented Oct 25, 2023

For the list-of-strings, parameters themselves can only contain bare items, not inner lists. https://www.rfc-editor.org/rfc/rfc8941.html#section-3.1.2

tfpauly added a commit that referenced this issue Oct 25, 2023
@ekline
Copy link

ekline commented Oct 25, 2023

👍

pmeenan pushed a commit to pmeenan/http-extensions that referenced this issue Nov 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

2 participants