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.

Sign up for GitHub

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

Jump to bottom

Investigate G602 bypass fix #2065

Open

jpmcb opened this issue Nov 4, 2023 · 0 comments

Collaborator

jpmcb commented Nov 4, 2023

cobra/command.go

Lines 780 to 782 in a0a6ae0

    
           // Temporarily disable gosec G602, which produces a false positive. 
        
           // See https://github.com/securego/gosec/issues/1005. 
        
           return matches[0] // #nosec G602

It appears there are some fixes for these // #nosec linter bypasses: securego/gosec#1017

We should investigate if we can consume these and not have to bypass this lint

The text was updated successfully, but these errors were encountered:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment