NotificationsYou must be signed in to change notification settings
Fork905
Star10k

fix: add detailed gosec G115 annotations#17085

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

Jump to bottom

Closed

sreya wants to merge1 commit intomainfromsreya/lint-fixes-new

Closed

fix: add detailed gosec G115 annotations#17085

sreya wants to merge1 commit intomainfromsreya/lint-fixes-new

Conversation

Copy link

Collaborator

sreya commentedMar 25, 2025

This PR adds more detailed #nosec G115 annotations to fix gosec linting issues in various files. These annotations explain why each integer conversion is safe and how potential overflow is handled.

The changes include:

Added explanatory comments for each integer conversion in several files
Documented specific cases where overflow is acceptable
Added bounds checking context where needed

These changes should resolve the remaining lint issues in#17035.

fix: add gosec G115 annotations

61110cb

Added more detailed #nosec G115 annotations to fix gosec warningsSigned-off-by: sreya <sreya@coder.com>

Copy link

github-actionsbot commentedMar 25, 2025

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign ourContributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

Claude seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, pleaseadd the email address used for this commit to your account.
_{You can retrigger this bot by commentingrecheck in this Pull Request.}_{Posted by theCLA Assistant Lite bot.}