pulibrary / recap

ReCAP Drupal 10 Site
GNU General Public License v2.0
2 stars 1 forks source link

Set the cookie SameSite directive to Strict #263

Closed sandbergja closed 10 months ago

sandbergja commented 10 months ago

Success criteria

Rationale

This reduces the number of scenarios in which the cookie will be sent, providing better CSRF protection and reducing the size of the Cookies header in some cases. Part of https://github.com/pulibrary/dacs_handbook/issues/154

Drupal 10 Documentation

Tradeoff

If a user logs into Recap, navigates to a non-Princeton Web site (e.g. Outlook), and then clicks a link from that Web site back to Recap, they will need to log in to Recap again.

Further reading

I found this from Mozilla, this from Portswigger, and this blog post from Andrew Lock to be helpful in understanding this topic.

sandbergja commented 10 months ago

Closing in favor of https://github.com/pulibrary/princeton_ansible/issues/4585