-
Notifications
You must be signed in to change notification settings - Fork 2.2k
Description
Go version 1.22 introduced significant changes to the pattern syntax and matching
behavior of http.ServerMux
which causes a large number of legacy unit tests to break.
(See https://pkg.go.dev/net/http#hdr-Compatibility-ServeMux for more information.)
As a result, testing of this repo is currently performed by setting this env variable:
export GODEBUG=httpmuxgo121=1
It would be greatly appreciated if an experienced Go developer would update this repo's
legacy unit tests so that go test ./...
passes without the use of setting the GODEBUG
environment variable.
Ideally, no unit tests would need to be deleted, however it is possible that some tests
no longer make sense. If that is the case, it would be nice to see a detailed explanation
for any removed unit tests as to why it no longer makes sense to keep.
Thank you in advance for your contributions! See CONTRIBUTING.md for helpful details
for contributing to this repo.