[chore] Remove unnecessary usage of nolint:gosec

[zzzk1]

Description

• Remove unnecessary usage of // nolint:gosec introduced by PR fix(deps): update module github.com/golangci/golangci-lint to v1.61.0 #10935
• Add exclude-rules G115

Link to tracking issue

Fixes #11425

Testing

Documentation

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 91.67%. Comparing base (c14d4f9) to head (de25735).

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #12112      +/-   ##
==========================================
- Coverage   91.67%   91.67%   -0.01%     
==========================================
  Files         461      461              
  Lines       24749    24740       -9     
==========================================
- Hits        22689    22680       -9     
  Misses       1678     1678              
  Partials      382      382              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[bogdandrutu]

Remove incorrect usage of // nolint:gosec introduced by PR #10935

I don't think they were incorrect, but the rules were relaxed.

[bogdandrutu]

No need for chlog here.

[zzzk1]

Remove incorrect usage of // nolint:gosec introduced by PR #10935

I don't think they were incorrect, but the rules were relaxed.

So, all we need to do is add exclude rules and then ignore them, right?

[bogdandrutu]

I am confused, I did not see that you excluded it.

[bogdandrutu]

I missed that you excluded. We should not exclude the rule, but fix the usage.

[zzzk1]

@bogdandrutu G115 has not been fixed yet: https://golangci-lint.run/product/changelog/. I tried using a condition like if a <= math.MaxInt64() (where a is of type uint64), but it did not work.