Skip to content
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

Chore: Setting go versions to be current #226

Merged

Conversation

MovieStoreGuy
Copy link
Collaborator

Context

Setting the versions to reflect the current LTS of go and previous version.

Changes

  • Updating CI go versions
  • Updated go mod verison to 1.22+
  • Ran mod tidy.

Due to the update in path matching in the http mux beyond 1.21, it means
that `%2f` is treated differently in path matching. To address this
within tests, the wildcard is used in its place
@MovieStoreGuy
Copy link
Collaborator Author

There is a fun change that happened as part of 1.21, the default mux changed how it handles path and pattern matching meaning the %2f is treated as / when handling paths.

The workaround was to add wildcards and that has worked for the tests.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added some additional changes here to use require instead of assert, since it would cause a panic on nil error.

@MovieStoreGuy MovieStoreGuy merged commit ebdaa3e into signalfx:master Oct 24, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants