Skip to content

JS: add CWE-532 to the js/clear-text-logging query #7103

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

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

erik-krogh
Copy link
Contributor

@erik-krogh erik-krogh requested a review from a team as a code owner November 10, 2021 13:16
@github-actions github-actions bot added the JS label Nov 10, 2021
@erik-krogh erik-krogh added the no-change-note-required This PR does not need a change note label Nov 10, 2021
@erik-krogh erik-krogh merged commit 0bf055f into github:main Nov 11, 2021
RasmusWL added a commit to RasmusWL/codeql that referenced this pull request Nov 24, 2021
Relevant for this query:

CWE-532: Insertion of Sensitive Information into Log File

> While logging all information may be helpful during development
> stages, it is important that logging levels be set appropriately
> before a product ships so that sensitive user data and system
> information are not accidentally exposed to potential attackers.

See https://cwe.mitre.org/data/definitions/532.html

JS also did this recently: github#7103
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
JS no-change-note-required This PR does not need a change note
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants