gbv / login-server

Login and connect accounts with multiple identity providers
https://coli-conc.gbv.de/login/
MIT License
31 stars 32 forks source link

CSP issues on /sessions page #92

Closed stefandesu closed 2 years ago

stefandesu commented 2 years ago

The "remove" and "Remove all other sessions" buttons on the /sessions page don't work due to CSP issues. This is because script-src even if configured with a nonce does not allow event handlers like onclick on elements directly. Instead, they need to be added via JavaScript using addEventListener. (There is a newer script-src-attr CSP header, but it is only supported in Chromium as of right now.)

Firefox also complains about the script-src-attr header which is set to none by default in Helmet.