Skip to content

Commit 0232646

Browse files
committed
Suggestions from code review
1 parent 0e7cbbf commit 0232646

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

java/ql/src/Security/CWE/CWE-522/InsecureBasicAuth.qhelp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
<qhelp>
33

44
<overview>
5-
<p>Basic authentication only obfuscates usernames and passwords in Base64 encoding, which can be easily recognized and reversed, thus it must not be transmitted over the cleartext HTTP channel. Transmission of sensitive information not in HTTPS is vulnerable to packet sniffing.</p>
5+
<p>Basic authentication only obfuscates usernames and passwords in Base64 encoding, which can be easily recognized and reversed, thus it must not be transmitted over the cleartext HTTP channel. Transmitting sensitive information without using HTTPS makes the data vulnerable to packet sniffing.</p>
66
</overview>
77

88
<recommendation>

java/ql/src/Security/CWE/CWE-522/InsecureBasicAuth.ql

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,8 @@
22
* @name Insecure basic authentication
33
* @description Basic authentication only obfuscates username/password in
44
* Base64 encoding, which can be easily recognized and reversed.
5-
* Transmission of sensitive information not over HTTPS is
6-
* vulnerable to packet sniffing.
5+
* Transmitting sensitive information without using HTTPS makes
6+
* the data vulnerable to packet sniffing.
77
* @kind path-problem
88
* @problem.severity warning
99
* @precision medium

0 commit comments

Comments
 (0)