Skip to content

Commit 485ee5c

Browse files
authored
Merge pull request #18692 from jcogs33/jcogs33/spring-csrf-qhelp-update
Java: update `java/spring-disabled-csrf-protection` QHelp
2 parents bc6ce32 + dce89c5 commit 485ee5c

File tree

1 file changed

+16
-6
lines changed

1 file changed

+16
-6
lines changed

java/ql/src/Security/CWE/CWE-352/SpringCSRFProtection.qhelp

+16-6
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,21 @@
22
<qhelp>
33

44
<overview>
5-
<p>When you set up a web server to receive a request from a client without any mechanism
6-
for verifying that it was intentionally sent, then it is vulnerable to attack. An attacker can
7-
trick a client into making an unintended request to the web server that will be treated as
8-
an authentic request. This can be done via a URL, image load, XMLHttpRequest, etc. and can
9-
result in exposure of data or unintended code execution.</p>
5+
<p>
6+
Cross-site request forgery (CSRF) is a type of vulnerability in which an
7+
attacker is able to force a user to carry out an action that the user did
8+
not intend.
9+
</p>
10+
11+
<p>
12+
The attacker tricks an authenticated user into submitting a request to the
13+
web application. Typically, this request will result in a state change on
14+
the server, such as changing the user's password. The request can be
15+
initiated when the user visits a site controlled by the attacker. If the
16+
web application relies only on cookies for authentication, or on other
17+
credentials that are automatically included in the request, then this
18+
request will appear as legitimate to the server.
19+
</p>
1020
</overview>
1121

1222
<recommendation>
@@ -26,7 +36,7 @@ by non-browser clients.</p>
2636
<references>
2737
<li>
2838
OWASP:
29-
<a href="https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)">Cross-Site Request Forgery (CSRF)</a>.
39+
<a href="https://www.owasp.org/index.php/Cross-Site_Request_Forgery_(CSRF)">Cross Site Request Forgery (CSRF)</a>.
3040
</li>
3141
<li>
3242
Spring Security Reference:

0 commit comments

Comments
 (0)