search

mrkkrp / modern-uri

Modern library for working with URIs
Other
68 stars 18 forks source link

Restrict port to max allowed size (65535) #22

Closed nghamilton closed 5 years ago

nghamilton commented 5 years ago

Changed Word to Word16.

mrkkrp commented 5 years ago

Hmm, I don't see where in the spec it's said that max value for port number is 65535.

nghamilton commented 5 years ago

Oh, right! I can't argue with Bernes-Lee. Sorry to waste your time.

On Wed, 23 Jan 2019 at 19:17 Mark Karpov notifications@github.com wrote:

Hmm, I don't see where in the spec https://tools.ietf.org/html/rfc3986#section-3.2.3 it's said that max value for port number is 65535.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/mrkkrp/modern-uri/pull/22#issuecomment-456727416, or mute the thread https://github.com/notifications/unsubscribe-auth/AIk3jZGwgg_ON2HdBqwqeuIzl6_GXgAAks5vGCi3gaJpZM4aN2VS .

mrkkrp commented 5 years ago

No problem!