- Notifications
You must be signed in to change notification settings - Fork907
chore: fix gosec G115 linting issues for Go 1.24.1 update#17084
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
Added more detailed #nosec G115 annotations to fix gosec warnings🤖 Generated with [Claude Code](https://claude.ai/code)Co-Authored-By: Claude <noreply@anthropic.com>
- Fix go:build directive spacing in pty_linux.go- Add bounds checks and #nosec annotations for integer conversions- Fix comment alignment and formatting- Address gosec G115 warnings in multiple filesCo-Authored-By: Claude <noreply@anthropic.com>
I have read the CLA Document and I hereby sign the CLA 0 out of2 committers have signed the CLA. |
Adds detailed #nosec G115 annotations to resolve gosec linting issues for the Go 1.24.1 upgrade in PR#17035.
This PR adds proper annotations with detailed comments to explain why each integer conversion is safe, which helps the linter understand that these conversions are intentional and the potential overflow risks are handled appropriately.
Part of the work for#17035.