Skip to content

[Vaultwarden] Hint at problems with default Web Security Headers #1902

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[Vaultwarden] Hint at problems with default Web Security Headers #1902

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions source/guide_vaultwarden.rst
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -123,6 +123,10 @@ Setup web backend

If you want to use a subdomain refer to the :manual:`web-backend manual <web-backends>`.

.. note::
The default nginx config defines `X-Frame-Options: SAMEORIGIN` for all web backends. This may cause compatibility issues with the 2-step login feature and desktop/mobile clients as well as notifications on websockets.
If you encounter these issues, you may want to consider suppressing that header for relevant paths i. e. /*connector.html, /notifications/hub, /notifications/anonymous-hub, refer to the :manual:`web-security-headers manual<web-security-headers>`.
Copy link
Member

@nichtmax nichtmax Jun 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It would be nice if you told how exactly to suppress these headers in this case.

Copy link
Member

@SalocinHB SalocinHB Aug 8, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@JayBraker any updates on this? If not, we'll close this PR soon.

Copy link
Author

@JayBraker JayBraker Aug 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank your for the reminder.
I amended my proposal for a snippet/codeblock and rephrased the note to make a little more sense in combination with the snippet.

Copy link
Member

@nichtmax nichtmax Oct 21, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

um.. did you? 😅 I can't find any additional commit. Could you please check?


Setup web vault
---------------

Expand Down