-
-
Notifications
You must be signed in to change notification settings - Fork 7.7k
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
Add support for Go's net/url JoinPath #9694
Comments
I'm pretty sure the above isn't true, but you are right, we are going to use the new JoinPath, but assuming it's not backported to Go 1.18, I suspect we need to wait until Go 1.20. |
For the record, even though Go 1.18 shipped in March, the feature freeze for it was back in November. It's part of the Go release cycle: https://github.com/golang/go/wiki/Go-Release-Cycle#timeline I actually did just barely miss the window for adding http.MaxBytesError to 1.18 because I wasn't paying attention to the freeze date. |
This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
I ran into an issue trying to use
path.Join
to join a path to a Permalink (I'm new to Hugo and Go):path.Join
usesfilepath.Join
under the hood, which callsfilepath.Clean
. Among other things,Clean
will remove any consecutive slashes, clobberinghttps://
in.Permalink
, making it for examplehttps:/example.com/path/to/a/thing
. Go'spath.Join
also does the same thing with its own implementation ofClean
.I dug around and found there's a very recently added (March 9th)
JoinPath
function innet/url
. I think it may have missed Go 1.18 by 5 days: it would be a convenient addition to Hugo on the next release of Go.The text was updated successfully, but these errors were encountered: