« BackCross-Site Request Forgerywords.filippo.ioSubmitted by tatersolid a day ago
  • MajesticHobo2 a day ago

    Not sure I agree with this part:

    > Allow all GET, HEAD, or OPTIONS requests.

    > These are safe methods, and are assumed not to change state at various layers of the stack already.

    Plenty of apps violate this assumption and do allow GET requests to alter state.

    • chrisfosterelli a day ago

      IMO apps that do this have a bug, and possibly a security one. This causes issues with prefetching, bot traffic, caching, CSRF, and just plain violates HTTP standards.

      • pinoy420 18 hours ago

        Not really. If I have a service where I need one click to perform an action and store data. It has to be a GET. You can’t post from a url… purist dogma for the sake of purist dogma

        • FireInsight 8 hours ago

          One click to perform an action and store data? Have you heard of HTML forms with method="post"?

          • whatabtemaillnk 5 hours ago

            [dead]

        • pstuart a day ago

          Agreed. Those methods should be treated as idempotent.

          • almog a day ago

            > Those methods should be treated as idempotent

            Idempotency still implies it can change state on the initial call, which to me feels wrong in the context of GET/HEAD/OPTIONS.

            • Mavvie a day ago

              Indeed, the correct term here is nullipotent.

        • cryptonector 19 hours ago

          This is on the server side, on the app. If your supposedly-safe methods aren't safe, then CSRF may not be your biggest problem.

          • simonw a day ago

            Those apps are beyond helping already. They need to fix theselves.

            • motorest 11 hours ago

              > Plenty of apps violate this assumption and do allow GET requests to alter state.

              Yeah, that's not a justification. From a RESTful API design perspective, this just means plenty of apps are buggy/critical design problems. A bug in a random app does not mean HTTP verb lose their semantics.

              • paulhodge 18 hours ago

                That’s bad because visiting an evil site can easily trick your browser into performing one of those requests using your own credentials. CORS doesn’t stop the backend state effect from happening.

                • MajesticHobo2 3 hours ago

                  That's exactly why I don't agree that GETs should be broadly exempted from CSRF protections. I'm not talking about CORS at all.

                • nchmy a day ago

                  The entire WordPress ecosystem says hello

                • akersten a day ago

                  I'm not really grokking the explanation in the article of why the SameSite cookie attribute doesn't fix CSRF. I thought that was the whole design intent of SameSite=Secure on an HTTPS cookie, was to fix CSRF. Can someone boil it down?

                  The article seemingly says "these cookies won't be sent with an unsafe request. But that doesn't fix it!" And doesn't elaborate?

                  • MajesticHobo2 a day ago

                    The problem boils down to the lack of equivalence between a site and an origin. The article explains how https://app.example.com and https://marketing.example.com may sit at very different trust levels, but are considered the same site by the browser. You don't want https://marketing.example.com to be able to make requests to https://app.example.com with your authentication cookies, but SameSite wouldn't prevent that.

                    • edoceo a day ago

                      This doesn't match my experience. What am I doing different?

                      Example I set SameSite=Strict on www.edoceo.com and then visiting app.edoceo.com the cookie is not there? They are different sites, different origins. And the cookie is set to the domain (ie: host, ie: www.edoceo.com)

                      • FiloSottile a day ago

                        For CSRF (and for SameSite), you are not looking at what cookies are sent to attacker.example.com, but what cookies are sent to target.example.com if a request is originated from attacker.example.com (or from attacker.com).

                      • undefined a day ago
                        [deleted]
                      • FiloSottile a day ago

                        Same-Site cookies are, well, same-site. Not same-origin. This is already a deal-breaker for many deployments, because they don't trust blog.example.com and partner.example.com as much as admin.example.com (both in the strict sense of trust, and in the senso of not having XSS vulnerabilities the attacker can pivot off).

                        Worse, by the original definition http://foo.example.com and https://admin.example.com are same-site, and unless the site uses HSTS with includeSubDomains, any network attacker controls the former. Chrome changed that with Schemeful Same-Site in 2020, but Firefox and Safari never deployed it.

                        • pluto_modadic 20 hours ago

                          ah, that explains it.

                        • harg 15 hours ago

                          The other replies answer this question, but it’s worth mentioning the public suffix list which contains a list of domain suffixes that have subdomains that are controlled by different people. E.g github.io, wordpress.com

                          Browser use this list to prevent cookie shared between sites using the suffixes on the list. E.g evil.github.io will not receive cookies from nice.github.io, or any other .github.io origin, regardless of the SameSite attribute

                          • undefined a day ago
                            [deleted]
                          • AgentME a day ago

                            It's very nice to have an up-to-date writeup like this. I've gotten some odd looks for telling people that classic CSRF tokens are unnecessary work since the Origin header became widely supported, and I'm glad to have a page like this to refer people to.

                          • fabian2k a day ago

                            So am I understanding it right that you don't need any CSRF tokens anymore to fully protect against CSRF attacks?

                            And if Go is implementing this specific protection, are other ecosystems doing this as well? My specific interest would be .NET/C#, but I am wondering in general how widespread this specific solution is at the moment.

                            • caleblloyd 11 hours ago

                              I don’t quite understand the part of the article that deems that you can skip all the checks under the assumption that this is an older browser, and that there is no CSRF vulnerability.

                              The algorithm seems sane for modern browsers. But you could probably find an outdated browser - older Android device WebView would be common -where the whole thing breaks down.

                              So I think tokens can be a thing of the past for modern browsers. I like the middleware, I hope it does show up in ASP.NET proper soon. My guess is they’ll keep tokens middleware around alongside it for some time once it does though, and the decision on which to use will come down to whether or not you want to make sure older browsers are secure.

                            • nchmy a day ago

                              i just discovered the Sec-Fetch stuff recently, due to Go 1.25's changelog. Very excited to start using it in some applications where tokens are currently used - what a hassle to deal with those.

                              • jerf a day ago

                                Cookies have been truly horrible. I check in on them every couple of years, because I don't do a lot of front-end but when I do it's often security-sensitive, and every single time I check in on them there's some new entry in "SameSite; NoSeriouslySecureHarder; WhoopsTheLastStandardWasNotGoodEnough=BeActuallySecure; AwwShitDidWeGetItRightLastTime=false" parade of attributes you need to send to get actually secure cookies.

                                No shade on the people implementing this stuff, I understand the backwards compatibility concerns, but I mean, keeping up with this stuff is harder than it should be. And thanks to backwards compatibility most of it still defaults open, though browsers have pecked at that as they can.

                                • unethical_ban a day ago

                                  Your examples made me chuckle. I was thinking "God I hope frameworks deal with all this stuff".

                                  • pstuart a day ago

                                    Fortunately the stability and consistency of JS frameworks make light work of that pain!

                                • undefined a day ago
                                  [deleted]
                                  • cryptonector 19 hours ago

                                    > Essentially all applications that use cookies for authentication need to protect against CSRF.

                                    Not just cookies!! Any HTTP authentication. Kerberos, NTLM, OAuth.

                                    • todotask2 17 hours ago

                                      > Same site vs same site vs same origin

                                      I'm lost here.

                                      • dizlexic a day ago

                                        [flagged]

                                        • undefined a day ago
                                          [deleted]