net/http: optimize Cookie.sanitizeOrWarn and timeoutHandler.ServeHTTP#71121
net/http: optimize Cookie.sanitizeOrWarn and timeoutHandler.ServeHTTP#71121eudore wants to merge 1 commit intogolang:masterfrom
Conversation
Cookie.sanitizeOrWarn: Avoid restarting validation from the beginning when encountering invalid characters. timeoutHandler.ServeHTTP: Use a single channel for both panic recovery and done signaling.
|
This PR (HEAD: 2f70324) has been imported to Gerrit for code review. Please visit Gerrit at https://go-review.googlesource.com/c/go/+/640335. Important tips:
|
|
Message from Gopher Robot: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Ian Lance Taylor: Patch Set 1: Commit-Queue+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Go LUCI: Patch Set 1: Dry run: CV is trying the patch. Bot data: {"action":"start","triggered_at":"2025-01-04T16:46:44Z","revision":"23a7117a0e7284889aa1ea0f94a5e7b05c1d685e"} Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Ian Lance Taylor: Patch Set 1: -Commit-Queue Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Go LUCI: Patch Set 1: This CL has passed the run Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Go LUCI: Patch Set 1: LUCI-TryBot-Result+1 Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
|
Message from Damien Neil: Patch Set 1: (1 comment) Please don’t reply on this GitHub thread. Visit golang.org/cl/640335. |
Cookie.sanitizeOrWarn: Avoid restarting validation from the beginning when encountering invalid characters.
timeoutHandler.ServeHTTP: Use a single channel for both panic recovery and done signaling.