-
Notifications
You must be signed in to change notification settings - Fork 18.3k
path: add more examples for path.Clean #74397
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
base: master
Are you sure you want to change the base?
Conversation
This PR (HEAD: 114519f) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/684376. Important tips:
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Gopher Robot: Patch Set 1: Congratulations on opening your first change. Thank you for your contribution! Next steps: Most changes in the Go project go through a few rounds of revision. This can be During May-July and Nov-Jan the Go project is in a code freeze, during which Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Rob Pike: Patch Set 2: (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
This PR (HEAD: d669e1e) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/684376. Important tips:
|
Message from Sean Liao: Patch Set 3: Commit-Queue+1 (2 comments) Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Go LUCI: Patch Set 3: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-07-06T03:00:19Z","revision":"62bdb2ac0c8dfe070baede2cf65438e4a92f126b"} Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Sean Liao: Patch Set 3: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Go LUCI: Patch Set 3: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Go LUCI: Patch Set 3: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Message from Rob Pike: Patch Set 4: Code-Review+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/684376. |
Clarify that the function path.Clean only normalises paths and does not
protect against directory-traversal attacks.