Hi Erik, Thanks for the review! I opened this issue: https://github.com/httpwg/http-extensions/issues/2667 And this PR: https://github.com/httpwg/http-extensions/pull/2668 Responses inline. Best, Tommy > On Oct 20, 2023, at 11:13 PM, Erik Kline via Datatracker <noreply@ietf.org> wrote: > > Erik Kline has entered the following ballot position for > draft-ietf-httpbis-alias-proxy-status-05: No Objection > > When responding, please keep the subject line intact and reply to all > email addresses included in the To and CC lines. (Feel free to cut this > introductory paragraph, however.) > > > Please refer to https://www.ietf.org/about/groups/iesg/statements/handling-ballot-positions/ > for more information about how to handle DISCUSS and COMMENT positions. > > > The document, along with other ballot positions, can be found here: > https://datatracker.ietf.org/doc/draft-ietf-httpbis-alias-proxy-status/ > > > > ---------------------------------------------------------------------- > COMMENT: > ---------------------------------------------------------------------- > > # Internet AD comments for draft-ietf-httpbis-alias-proxy-status-05 > CC @ekline > > * comment syntax: > - https://github.com/mnot/ietf-comments/blob/main/format.md > > * "Handling Ballot Positions": > - https://ietf.org/about/groups/iesg/statements/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)? Parameters themselves can only contain bare items, not inner lists. (See https://www.rfc-editor.org/rfc/rfc8941.html#section-3.1.2) > > ### S3 > > * Up to you, but feel free to reference RFC 3493 section 6.1 for getaddrinfo > and discussion of the AI_CANONNAME flag. Sounds good! Opened the PR for that. > > ## Nits > > ### S3 > > * "might not available" -> > "might not be available" Thanks! The nit was fixed with https://github.com/httpwg/http-extensions/pull/2647 . > > >
Received on Wednesday, 25 October 2023 15:52:00 UTC