- Notifications
You must be signed in to change notification settings - Fork43
update port references#246
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
reschke left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I believe we need more details here: it sounds like this should updateall entries for 80 and 443? What is the exact text that should go here? Also, I guess this is a change from RFC723* (and before), as they did not register ports at all?
mnot commentedSep 3, 2019
No, all entries for 80 and 443 that use TCP or UDP. I have no opinion about SCTP, and don't want to take over authority for it in these specs. I think the instruction in the PR is sufficient; all we're doing is changing (or adding) references. |
reschke commentedSep 3, 2019
My experience with IANA is that they want precise instructions. It also helps readers/reviewers understand what's happening. |
royfielding left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I think this is correct, or at least better than the current mess at IANA.
reschke left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
I'm fine with the intent, but I believe we need to be more precise. I can make a proposal.
reschke commentedSep 5, 2019 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
See8835112 - which begs the question whether we should also update assignee/contact for TCP/UDP "www-http" which are currently assigned to TimBL. (I also note that these are present for but not for 443). |
mnot commentedSep 9, 2019
Works for me.@reschke you now OK? My instinct is to leave www-http alone. |
Fixes#36.