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

According to RFC5849 port number must be excluded from base string if it is the default #59

Open
orontee opened this issue Sep 12, 2017 · 4 comments

Comments

@orontee
Copy link

orontee commented Sep 12, 2017

3.4.1.2. Base String URI
(...)
3. The port MUST be included if it is not the default port for the
scheme, and MUST be excluded if it is the default. Specifically,
the port MUST be excluded when making an HTTP request [RFC2616]
to port 80 or when making an HTTPS request [RFC2818] to port 443.
All other non-default port numbers MUST be included.

@orontee orontee changed the title According to RFC5849 port number must be excluded if it is the default According to RFC5849 port number must be excluded from basic string if it is the default Sep 12, 2017
@orontee orontee changed the title According to RFC5849 port number must be excluded from basic string if it is the default According to RFC5849 port number must be excluded from base string if it is the default Sep 12, 2017
@ddo
Copy link
Owner

ddo commented Sep 13, 2017

you have issues with custom port (not 80 and 443) ?

@orontee
Copy link
Author

orontee commented Sep 13, 2017 via email

@ddo
Copy link
Owner

ddo commented Sep 13, 2017

so if you input http://example.com:80 or https://example.com:443 it gonna be an issue?

@orontee
Copy link
Author

orontee commented Sep 13, 2017 via email

dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Dec 22, 2017
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Dec 23, 2017
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Dec 24, 2017
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Dec 26, 2017
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Jan 3, 2018
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Jan 3, 2018
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Jan 3, 2018
dinoboff added a commit to dinoboff/oauth-1.0a that referenced this issue Jan 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants