Skip to content

Commit d233cea

Browse files
author
Max Schaefer
committed
JavaScript: Lower precision of PasswordInConfigurationFile.
In spite of recent improvements, this query is still too noisy to show by default.
1 parent d723ab7 commit d233cea

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

change-notes/1.21/analysis-javascript.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,7 @@
4040
| Expression has no effect | Fewer false-positive results | This rule now treats uses of `Object.defineProperty` more conservatively. |
4141
| Incomplete regular expression for hostnames | More results | This rule now tracks regular expressions for host names further. |
4242
| Incomplete string escaping or encoding | More results | This rule now considers the flow of regular expressions literals, and it no longer flags the removal of trailing newlines. |
43-
| Password in configuration file | Fewer false positive results | This query now excludes passwords that are inserted into the configuration file using a templating mechanism or read from environment variables. |
43+
| Password in configuration file | Fewer false positive results | This query now excludes passwords that are inserted into the configuration file using a templating mechanism or read from environment variables. Results are no longer shown on LGTM by default. |
4444
| Replacement of a substring with itself | More results | This rule now considers the flow of regular expressions literals. |
4545
| Server-side URL redirect | Fewer false-positive results | This rule now treats URLs as safe in more cases where the hostname cannot be tampered with. |
4646
| Type confusion through parameter tampering | Fewer false-positive results | This rule now recognizes additional emptiness checks. |

javascript/ql/src/Security/CWE-313/PasswordInConfigurationFile.ql

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
* @description Storing unencrypted passwords in configuration files is unsafe.
44
* @kind problem
55
* @problem.severity warning
6-
* @precision high
6+
* @precision medium
77
* @id js/password-in-configuration-file
88
* @tags security
99
* external/cwe/cwe-256

0 commit comments

Comments
 (0)