Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

codeql false positives #86

Closed
PietroPasotti opened this issue Apr 29, 2024 · 1 comment
Closed

codeql false positives #86

PietroPasotti opened this issue Apr 29, 2024 · 1 comment

Comments

@PietroPasotti
Copy link
Contributor

Bug Description

Codeql gets triggered by some debug logging in cert_handler, resulting in CI failure and noctua being unable to merge automatic charm lib updates.

See for example in traefik:
canonical/traefik-k8s-operator#342

image

To Reproduce

canonical/traefik-k8s-operator#342

Environment

n/a

Relevant log output

n/a

Additional context

We should investigate marking this lines as noqa or something like that, or renaming the variables before printing them so codeql won't get touchy about it.

@ca-scribner
Copy link
Contributor

Closed as it'll soon be invalid when we move to Tiobe

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

No branches or pull requests

2 participants