Skip to content

Update unit tests so that GODEBUG=httpmuxgo121=1 is not required #3409

@gmlewis

Description

@gmlewis

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    challengingThis issue is for experienced Go developers only, please.enhancementgoPull requests that update Go code

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions