chore: Document godebug flag usage #3429
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
sfc-gh-jcieslak
force-pushed
the
document-godebug-flag-usage
branch
from
815eae5 to
73ba109
Compare
|
Integration tests cancelled for 73ba1093e0650f605c2aca7291247f6964066654 |
There was a problem hiding this comment.
PR Overview
This PR documents the usage of the GODEBUG flag as a workaround for Go standard library changes that affect compatibility with Suricata-based firewalls, as described in issue #3421.
- Adds documentation in MIGRATION_GUIDE.md for the new Go version (v1.23.6) and the GODEBUG workaround.
- Updates CREATING_ISSUES.md to reference Go-related issues and link to the migration guide for detailed instructions.
Reviewed Changes
| File | Description |
|---|---|
| MIGRATION_GUIDE.md | Added a new section to document the Go version bump and GODEBUG usage. |
| CREATING_ISSUES.md | Included a new bullet and detailed section on handling Go-related firewall issues. |
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
sfc-gh-jmichalak
approved these changes
Mar 3, 2025
sfc-gh-jmichalak
approved these changes
Mar 3, 2025
sfc-gh-asawicki
approved these changes
Mar 4, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Document issue #3421 and how the godebug environment variable can be used to turn off some of the Go features that may not always be compatible with third-party software (like firewalls).