We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
1 parent 2cc7033 commit ae63436Copy full SHA for ae63436
1 file changed
ruby/ql/src/experimental/manually-check-http-verb/ManuallyCheckHttpVerb.qhelp
@@ -0,0 +1,23 @@
1
+<!DOCTYPE qhelp PUBLIC
2
+"-//Semmle//qhelp//EN"
3
+"qhelp.dtd">
4
+<qhelp>
5
+ <overview>
6
+ <p>
7
+ Manually checking the HTTP request verb inside of a controller method can lead to
8
+ CSRF bypass if GET or HEAD requests are handled improperly.
9
+ </p>
10
+ </overview>
11
+ <recommendation>
12
13
+ It is better to use different controller methods for each resource/http verb combination
14
+ and configure the Rails routes in your application to call them accordingly.
15
16
+ </recommendation>
17
+
18
+ <references>
19
20
+ See https://guides.rubyonrails.org/routing.html for more information.
21
22
+ </references>
23
+</qhelp>
0 commit comments