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

Incorrect answer for question Who will be notified when a NEW secret is pushed and detected in a repository? #262

Closed
carlos-quintero opened this issue Aug 14, 2024 · 2 comments
Labels
advanced security Related to GitHub Advanced Security Exam content Related to the content of the site good first issue Good for newcomers help wanted Extra attention is needed

Comments

@carlos-quintero
Copy link

In practice exam Advanced Security, question 17

”Who will be notified when a NEW secret is pushed and detected in a repository? “

(https://github.com/FidelusAleksander/ghcertified/blob/master/content/questions/advanced_security/question-017.md)

the answer “Commit authors” should be included since incremental scans notify them. See:

Note

Commit authors who've accidentally committed secrets will be notified, regardless of their notification preferences.

Reference:

https://docs.github.com/en/code-security/secret-scanning/managing-alerts-from-secret-scanning/monitoring-alerts#incremental-scans

BTW the link provided in the question is incorrect, the correct is the one above.

@carlos-quintero carlos-quintero added the content Related to the content of the site label Aug 14, 2024
@FidelusAleksander
Copy link
Owner

Hi @carlos-quintero !

Seems you are right here. If you would like to update the answers, take a look at the contributing guide.
If not - I will fix it myself in the following days

Thanks for raising the issue 🚀

@FidelusAleksander FidelusAleksander added help wanted Extra attention is needed good first issue Good for newcomers advanced security Related to GitHub Advanced Security Exam labels Aug 16, 2024
@FidelusAleksander
Copy link
Owner

Done in #269

Thanks for raising the issue 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
advanced security Related to GitHub Advanced Security Exam content Related to the content of the site good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants