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
net/url: func parse(rawURL string, viaRequest bool) (*URL, error) #63280
Labels
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
Comments
bcmills
changed the title
affected/package: net/url func parse(rawURL string, viaRequest bool) (*URL, error)
net/url: func parse(rawURL string, viaRequest bool) (*URL, error)
Sep 28, 2023
RFC 1738 defines:
But |
bcmills
added
the
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
label
Sep 28, 2023
but it's an or operation |
The character |
bcmills
added
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
and removed
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
labels
Sep 28, 2023
I think the problem comes along with character ?, not : please check code
between line 531-536
https://github.com/golang/go/blob/master/src/net/url/url.go
…On Thu 28. Sep 2023 at 19:18, Bryan C. Mills ***@***.***> wrote:
The character : is not in that list.
—
Reply to this email directly, view it on GitHub
<#63280 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADCD6BGC24J4LO5JOPAZQ43X4WWQBANCNFSM6AAAAAA5KSA3DU>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
we use RFC 3986, not 1738 |
thanks! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
WaitingForInfo
Issue is not actionable because of missing required information, which needs to be provided.
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
https://go.dev/play/p/njAfwJZKrXZ
What did you expect to see?
https://www.rfc-editor.org/rfc/rfc1738.html
user = *[ uchar | ";" | "?" | "&" | "=" ]
password = *[ uchar | ";" | "?" | "&" | "=" ]
What did you see instead?
invalid port ":p" after host
The text was updated successfully, but these errors were encountered: