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

Writing IPv4 addresses #12

Open
annevk opened this issue Dec 9, 2014 · 4 comments
Open

Writing IPv4 addresses #12

annevk opened this issue Dec 9, 2014 · 4 comments
Labels

Comments

@annevk
Copy link
Contributor

annevk commented Dec 9, 2014

We should just require the syntax from RFC 3986, but instead of ABNF use prose to remain consistent. (Although I really would have expected us to use syntax diagrams here, as this section is aimed at web developers and not parser implementers.)

@rubys
Copy link
Member

rubys commented Dec 9, 2014

Is this missing a word? I'm either having difficultly parsing the part before the first comma or I'm having difficultly reconciling it with the remainder of the first sentence.

I'm presuming that this is concerning the <p class=XXX> concerning IPv4 addresses in what is currently called "Authoring Requirements", but likely will be renamed to something related to URL string conformance requirements.

@annevk
Copy link
Contributor Author

annevk commented Dec 9, 2014

It's not missing a word. I'm just saying that a valid IPv4 address should be one that is valid per RFC 3986, but we should define that in a way that is consistent with how we define the syntax of other parts of the URL.

@rubys
Copy link
Member

rubys commented Dec 9, 2014

Thanks for the clarification. Got it. Here's the ABNF:

     IPv4address = dec-octet "." dec-octet "." dec-octet "." dec-octet

      dec-octet   = DIGIT                 ; 0-9
                  / %x31-39 DIGIT         ; 10-99
                  / "1" 2DIGIT            ; 100-199
                  / "2" %x30-34 DIGIT     ; 200-249
                  / "25" %x30-35          ; 250-255

So exactly four numbers, all numbers are decimal, no leading zeros, all values between 0 and 255, ... things like that. Make sense to me.

@annevk
Copy link
Contributor Author

annevk commented Dec 9, 2014

Yup, that.

rubys added a commit that referenced this issue Dec 16, 2014
editors plural - editorial change
@rubys rubys added the WHATWG label Dec 29, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants