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

Har PR i sweden så denne trengs ikke merges. Database spider alert #428

Closed

Conversation

JohannesFinsveen
Copy link

@JohannesFinsveen JohannesFinsveen commented Nov 13, 2024

på alerten på database spider.
(Litt usannsynlig abUsecase her, de har alt fått tak i apikey, og gjør et kall med database "../blabla". Så gevinsten med å fikse dette det er vel mest at det ser stygt ut hvis noen laster ned koden og scanner, selv om det bare er false positiver)

@JohannesFinsveen JohannesFinsveen changed the title la inn forslaget fra ai Har PR i sweden så denne trengs ikke merges. Database spider alert Nov 13, 2024
@JohannesFinsveen
Copy link
Author

Nå er den i master i sweden

@JohannesFinsveen JohannesFinsveen deleted the Fearture/Codeql-alert-database-spider branch November 20, 2024 13:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant