-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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.
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
WebSockets via Rocket's Upgrade connection #3404
Conversation
e5a4bfb
to
e0203db
Compare
I think the implementation is working. It can use both port 3012 Direct Tungstenite and the Rocket/Tungstenite implementation. Best would be if Rocket 0.5 would be released as stable so that we do not have to use a patched version. |
b763c93
to
3e3b000
Compare
4a634b2
to
2382e12
Compare
2382e12
to
0cc9c48
Compare
This PR implements a (not yet fully released) new feature of Rocket which allows WebSockets/Upgrade connections. No more need for multiple ports to be opened for Vaultwarden. No explicit need for a reverse proxy to get WebSockets to work (Although I still suggest to use a reverse proxy). - Using a git revision for Rocket, since `rocket_ws` is not yet released. - Updated other crates as well. - Added a connection guard to clear the WS connection from the Users list. Fixes dani-garcia#685 Fixes dani-garcia#2917 Fixes dani-garcia#1424
0cc9c48
to
3d11f4c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me, do we want to wait until there's a stable release or a release candidate or should we just merge and allow it to be tested on the testing
images?
@dani-garcia , i think for testing it's fine. |
True, I don't expect this change to break anything, and we still have the 3012 fallback anyway. |
Just as a note here for now. If you want to use this, you still need to make sure that your reverse proxy pass-through the upgrade request correctly. For nginx you can use the following. Somewhere within the map $http_upgrade $connection_upgrade {
default upgrade;
'' "";
} And then within the location. location / {
proxy_http_version 1.1;
proxy_set_header Upgrade $http_upgrade;
proxy_set_header Connection $connection_upgrade; |
As per dani-garcia/vaultwarden#3404, we no longer need a dedicated websocket port.
With this change (WebSocket notifications now work via the default HTTP port ) should I remove this line from my docker compose :
thanks |
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [vaultwarden/server](https://togithub.com/dani-garcia/vaultwarden) | stage | minor | `1.28.1-alpine` -> `1.29.0-alpine` | --- ### Release Notes <details> <summary>dani-garcia/vaultwarden (vaultwarden/server)</summary> ### [`v1.29.0`](https://togithub.com/dani-garcia/vaultwarden/releases/tag/1.29.0) [Compare Source](https://togithub.com/dani-garcia/vaultwarden/compare/1.28.1...1.29.0) #### Major changes and New Features - WebSocket notifications now work via the default HTTP port. No need for `WEBSOCKET_ENABLED` and a separate port anymore. The proxy examples still need to be updated for this. Support for the old websockets port 3012 will remain for the time being. - Mobile Client push notification support, see [#​3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304) thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! - Web-Vault updated to v2023.5.0 (v2023.5.1 does not add any improvements for us) - The latest Bitwarden Directory Connector can be used now ([v2022.11.0](https://togithub.com/bitwarden/directory-connector/releases/tag/v2022.11.0)) - [Storing passkeys](https://bitwarden.com/passwordless-passkeys) is supported, though the clients are not yet released. So, it might be we need to make some changes once they are released. See: [#​3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593), thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! #### What's Changed - check if reset password policy is enabled by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3427](https://togithub.com/dani-garcia/vaultwarden/pull/3427) - WebSockets via Rocket's Upgrade connection by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3404](https://togithub.com/dani-garcia/vaultwarden/pull/3404) - Several config and admin interface fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3436](https://togithub.com/dani-garcia/vaultwarden/pull/3436) - Fixed missing footer_text and a few inconsistencies in email templates by [@​kennymc-c](https://togithub.com/kennymc-c) in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - Small update to Rocket WebSockets by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3440](https://togithub.com/dani-garcia/vaultwarden/pull/3440) - inline static rsa keys by [@​vilgotf](https://togithub.com/vilgotf) in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - Update Rust and Crates by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3469](https://togithub.com/dani-garcia/vaultwarden/pull/3469) - Change `String` to `&str` for all Rocket functions and some other fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3491](https://togithub.com/dani-garcia/vaultwarden/pull/3491) - Use Rocket `v0.5` branch to fix endpoints by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3502](https://togithub.com/dani-garcia/vaultwarden/pull/3502) - Use fully qualified image names in Dockerfile by [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - policy data should be `null` not an empty object by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3513](https://togithub.com/dani-garcia/vaultwarden/pull/3513) - update web-vault to v2023.4.2 by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3522](https://togithub.com/dani-garcia/vaultwarden/pull/3522) - Sync global_domains.json (Pinterest) by [@​jjlin](https://togithub.com/jjlin) in [https://github.com/dani-garcia/vaultwarden/pull/3532](https://togithub.com/dani-garcia/vaultwarden/pull/3532) - Prevent 401 on main admin page by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3547](https://togithub.com/dani-garcia/vaultwarden/pull/3547) - Update crates and GH Workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3548](https://togithub.com/dani-garcia/vaultwarden/pull/3548) - Fix collection change ws notifications by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3546](https://togithub.com/dani-garcia/vaultwarden/pull/3546) - Update Rust and Crates by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3563](https://togithub.com/dani-garcia/vaultwarden/pull/3563) - feat: Implement Push Notifications sync by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3304](https://togithub.com/dani-garcia/vaultwarden/pull/3304) - Implement the Organization API Key support for the new Directory Connector v2022 by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3568](https://togithub.com/dani-garcia/vaultwarden/pull/3568) - Add mobile push device filter to non-null push uuid by [@​quexten](https://togithub.com/quexten) in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - Update crates and workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3603](https://togithub.com/dani-garcia/vaultwarden/pull/3603) - Add group import on invite by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3606](https://togithub.com/dani-garcia/vaultwarden/pull/3606) - Fix send access regression by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3608](https://togithub.com/dani-garcia/vaultwarden/pull/3608) - feat: Support for storing passkeys in the vault by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3593](https://togithub.com/dani-garcia/vaultwarden/pull/3593) - add user to collection during creation by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3609](https://togithub.com/dani-garcia/vaultwarden/pull/3609) - Updated docker run command by [@​DenuxPlays](https://togithub.com/DenuxPlays) in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - Added-External_id for Collections by [@​fashberg](https://togithub.com/fashberg) in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) - fix missing password check while manual reset password enrollment by [@​sirux88](https://togithub.com/sirux88) in [https://github.com/dani-garcia/vaultwarden/pull/3632](https://togithub.com/dani-garcia/vaultwarden/pull/3632) - Update crates and small clippy fix by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3649](https://togithub.com/dani-garcia/vaultwarden/pull/3649) - fix version when compiled at a specific commit by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3651](https://togithub.com/dani-garcia/vaultwarden/pull/3651) - Fix org creation regresion by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659) #### New Contributors - [@​kennymc-c](https://togithub.com/kennymc-c) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - [@​vilgotf](https://togithub.com/vilgotf) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - [@​quexten](https://togithub.com/quexten) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - [@​DenuxPlays](https://togithub.com/DenuxPlays) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - [@​fashberg](https://togithub.com/fashberg) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) **Full Changelog**: dani-garcia/vaultwarden@1.28.1...1.29.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on saturday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/arthurgeek/vaultwarden-fly-template). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41LjMiLCJ1cGRhdGVkSW5WZXIiOiIzNi41LjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [vaultwarden/server](https://togithub.com/dani-garcia/vaultwarden) | stage | minor | `1.28.1-alpine` -> `1.29.0-alpine` | --- ### Release Notes <details> <summary>dani-garcia/vaultwarden (vaultwarden/server)</summary> ### [`v1.29.0`](https://togithub.com/dani-garcia/vaultwarden/releases/tag/1.29.0) [Compare Source](https://togithub.com/dani-garcia/vaultwarden/compare/1.28.1...1.29.0) #### Major changes and New Features - WebSocket notifications now work via the default HTTP port. No need for `WEBSOCKET_ENABLED` and a separate port anymore. The proxy examples still need to be updated for this. Support for the old websockets port 3012 will remain for the time being. - Mobile Client push notification support, see [#​3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304) thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! - Web-Vault updated to v2023.5.0 (v2023.5.1 does not add any improvements for us) - The latest Bitwarden Directory Connector can be used now ([v2022.11.0](https://togithub.com/bitwarden/directory-connector/releases/tag/v2022.11.0)) - [Storing passkeys](https://bitwarden.com/passwordless-passkeys) is supported, though the clients are not yet released. So, it might be we need to make some changes once they are released. See: [#​3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593), thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! #### What's Changed - check if reset password policy is enabled by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3427](https://togithub.com/dani-garcia/vaultwarden/pull/3427) - WebSockets via Rocket's Upgrade connection by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3404](https://togithub.com/dani-garcia/vaultwarden/pull/3404) - Several config and admin interface fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3436](https://togithub.com/dani-garcia/vaultwarden/pull/3436) - Fixed missing footer_text and a few inconsistencies in email templates by [@​kennymc-c](https://togithub.com/kennymc-c) in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - Small update to Rocket WebSockets by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3440](https://togithub.com/dani-garcia/vaultwarden/pull/3440) - inline static rsa keys by [@​vilgotf](https://togithub.com/vilgotf) in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - Update Rust and Crates by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3469](https://togithub.com/dani-garcia/vaultwarden/pull/3469) - Change `String` to `&str` for all Rocket functions and some other fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3491](https://togithub.com/dani-garcia/vaultwarden/pull/3491) - Use Rocket `v0.5` branch to fix endpoints by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3502](https://togithub.com/dani-garcia/vaultwarden/pull/3502) - Use fully qualified image names in Dockerfile by [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - policy data should be `null` not an empty object by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3513](https://togithub.com/dani-garcia/vaultwarden/pull/3513) - update web-vault to v2023.4.2 by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3522](https://togithub.com/dani-garcia/vaultwarden/pull/3522) - Sync global_domains.json (Pinterest) by [@​jjlin](https://togithub.com/jjlin) in [https://github.com/dani-garcia/vaultwarden/pull/3532](https://togithub.com/dani-garcia/vaultwarden/pull/3532) - Prevent 401 on main admin page by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3547](https://togithub.com/dani-garcia/vaultwarden/pull/3547) - Update crates and GH Workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3548](https://togithub.com/dani-garcia/vaultwarden/pull/3548) - Fix collection change ws notifications by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3546](https://togithub.com/dani-garcia/vaultwarden/pull/3546) - Update Rust and Crates by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3563](https://togithub.com/dani-garcia/vaultwarden/pull/3563) - feat: Implement Push Notifications sync by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3304](https://togithub.com/dani-garcia/vaultwarden/pull/3304) - Implement the Organization API Key support for the new Directory Connector v2022 by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3568](https://togithub.com/dani-garcia/vaultwarden/pull/3568) - Add mobile push device filter to non-null push uuid by [@​quexten](https://togithub.com/quexten) in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - Update crates and workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3603](https://togithub.com/dani-garcia/vaultwarden/pull/3603) - Add group import on invite by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3606](https://togithub.com/dani-garcia/vaultwarden/pull/3606) - Fix send access regression by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3608](https://togithub.com/dani-garcia/vaultwarden/pull/3608) - feat: Support for storing passkeys in the vault by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3593](https://togithub.com/dani-garcia/vaultwarden/pull/3593) - add user to collection during creation by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3609](https://togithub.com/dani-garcia/vaultwarden/pull/3609) - Updated docker run command by [@​DenuxPlays](https://togithub.com/DenuxPlays) in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - Added-External_id for Collections by [@​fashberg](https://togithub.com/fashberg) in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) - fix missing password check while manual reset password enrollment by [@​sirux88](https://togithub.com/sirux88) in [https://github.com/dani-garcia/vaultwarden/pull/3632](https://togithub.com/dani-garcia/vaultwarden/pull/3632) - Update crates and small clippy fix by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3649](https://togithub.com/dani-garcia/vaultwarden/pull/3649) - fix version when compiled at a specific commit by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3651](https://togithub.com/dani-garcia/vaultwarden/pull/3651) - Fix org creation regresion by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659) #### New Contributors - [@​kennymc-c](https://togithub.com/kennymc-c) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - [@​vilgotf](https://togithub.com/vilgotf) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - [@​quexten](https://togithub.com/quexten) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - [@​DenuxPlays](https://togithub.com/DenuxPlays) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - [@​fashberg](https://togithub.com/fashberg) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) **Full Changelog**: dani-garcia/vaultwarden@1.28.1...1.29.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on saturday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/arthurgeek/vaultwarden-fly-template). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41LjMiLCJ1cGRhdGVkSW5WZXIiOiIzNi41LjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
[![Mend Renovate](https://app.renovatebot.com/images/banner.svg)](https://renovatebot.com) This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [vaultwarden/server](https://togithub.com/dani-garcia/vaultwarden) | stage | minor | `1.28.1-alpine` -> `1.29.0-alpine` | --- ### Release Notes <details> <summary>dani-garcia/vaultwarden (vaultwarden/server)</summary> ### [`v1.29.0`](https://togithub.com/dani-garcia/vaultwarden/releases/tag/1.29.0) [Compare Source](https://togithub.com/dani-garcia/vaultwarden/compare/1.28.1...1.29.0) #### Major changes and New Features - WebSocket notifications now work via the default HTTP port. No need for `WEBSOCKET_ENABLED` and a separate port anymore. The proxy examples still need to be updated for this. Support for the old websockets port 3012 will remain for the time being. - Mobile Client push notification support, see [#​3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304) thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! - Web-Vault updated to v2023.5.0 (v2023.5.1 does not add any improvements for us) - The latest Bitwarden Directory Connector can be used now ([v2022.11.0](https://togithub.com/bitwarden/directory-connector/releases/tag/v2022.11.0)) - [Storing passkeys](https://bitwarden.com/passwordless-passkeys) is supported, though the clients are not yet released. So, it might be we need to make some changes once they are released. See: [#​3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593), thanks [@​GeekCornerGH](https://togithub.com/GeekCornerGH)! #### What's Changed - check if reset password policy is enabled by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3427](https://togithub.com/dani-garcia/vaultwarden/pull/3427) - WebSockets via Rocket's Upgrade connection by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3404](https://togithub.com/dani-garcia/vaultwarden/pull/3404) - Several config and admin interface fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3436](https://togithub.com/dani-garcia/vaultwarden/pull/3436) - Fixed missing footer_text and a few inconsistencies in email templates by [@​kennymc-c](https://togithub.com/kennymc-c) in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - Small update to Rocket WebSockets by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3440](https://togithub.com/dani-garcia/vaultwarden/pull/3440) - inline static rsa keys by [@​vilgotf](https://togithub.com/vilgotf) in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - Update Rust and Crates by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3469](https://togithub.com/dani-garcia/vaultwarden/pull/3469) - Change `String` to `&str` for all Rocket functions and some other fixes by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3491](https://togithub.com/dani-garcia/vaultwarden/pull/3491) - Use Rocket `v0.5` branch to fix endpoints by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3502](https://togithub.com/dani-garcia/vaultwarden/pull/3502) - Use fully qualified image names in Dockerfile by [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - policy data should be `null` not an empty object by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3513](https://togithub.com/dani-garcia/vaultwarden/pull/3513) - update web-vault to v2023.4.2 by [@​stefan0xC](https://togithub.com/stefan0xC) in [https://github.com/dani-garcia/vaultwarden/pull/3522](https://togithub.com/dani-garcia/vaultwarden/pull/3522) - Sync global_domains.json (Pinterest) by [@​jjlin](https://togithub.com/jjlin) in [https://github.com/dani-garcia/vaultwarden/pull/3532](https://togithub.com/dani-garcia/vaultwarden/pull/3532) - Prevent 401 on main admin page by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3547](https://togithub.com/dani-garcia/vaultwarden/pull/3547) - Update crates and GH Workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3548](https://togithub.com/dani-garcia/vaultwarden/pull/3548) - Fix collection change ws notifications by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3546](https://togithub.com/dani-garcia/vaultwarden/pull/3546) - Update Rust and Crates by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3563](https://togithub.com/dani-garcia/vaultwarden/pull/3563) - feat: Implement Push Notifications sync by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3304](https://togithub.com/dani-garcia/vaultwarden/pull/3304) - Implement the Organization API Key support for the new Directory Connector v2022 by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3568](https://togithub.com/dani-garcia/vaultwarden/pull/3568) - Add mobile push device filter to non-null push uuid by [@​quexten](https://togithub.com/quexten) in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - Update crates and workflow by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3603](https://togithub.com/dani-garcia/vaultwarden/pull/3603) - Add group import on invite by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3606](https://togithub.com/dani-garcia/vaultwarden/pull/3606) - Fix send access regression by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3608](https://togithub.com/dani-garcia/vaultwarden/pull/3608) - feat: Support for storing passkeys in the vault by [@​GeekCornerGH](https://togithub.com/GeekCornerGH) in [https://github.com/dani-garcia/vaultwarden/pull/3593](https://togithub.com/dani-garcia/vaultwarden/pull/3593) - add user to collection during creation by [@​farodin91](https://togithub.com/farodin91) in [https://github.com/dani-garcia/vaultwarden/pull/3609](https://togithub.com/dani-garcia/vaultwarden/pull/3609) - Updated docker run command by [@​DenuxPlays](https://togithub.com/DenuxPlays) in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - Added-External_id for Collections by [@​fashberg](https://togithub.com/fashberg) in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) - fix missing password check while manual reset password enrollment by [@​sirux88](https://togithub.com/sirux88) in [https://github.com/dani-garcia/vaultwarden/pull/3632](https://togithub.com/dani-garcia/vaultwarden/pull/3632) - Update crates and small clippy fix by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3649](https://togithub.com/dani-garcia/vaultwarden/pull/3649) - fix version when compiled at a specific commit by [@​tessus](https://togithub.com/tessus) in [https://github.com/dani-garcia/vaultwarden/pull/3651](https://togithub.com/dani-garcia/vaultwarden/pull/3651) - Fix org creation regresion by [@​BlackDex](https://togithub.com/BlackDex) in [https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659) #### New Contributors - [@​kennymc-c](https://togithub.com/kennymc-c) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3439](https://togithub.com/dani-garcia/vaultwarden/pull/3439) - [@​vilgotf](https://togithub.com/vilgotf) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3475](https://togithub.com/dani-garcia/vaultwarden/pull/3475) - [@​gitouche-sur-osm](https://togithub.com/gitouche-sur-osm) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3505](https://togithub.com/dani-garcia/vaultwarden/pull/3505) - [@​quexten](https://togithub.com/quexten) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3578](https://togithub.com/dani-garcia/vaultwarden/pull/3578) - [@​DenuxPlays](https://togithub.com/DenuxPlays) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3620](https://togithub.com/dani-garcia/vaultwarden/pull/3620) - [@​fashberg](https://togithub.com/fashberg) made their first contribution in [https://github.com/dani-garcia/vaultwarden/pull/3623](https://togithub.com/dani-garcia/vaultwarden/pull/3623) **Full Changelog**: dani-garcia/vaultwarden@1.28.1...1.29.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - "on saturday" (UTC), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Mend Renovate](https://www.mend.io/free-developer-tools/renovate/). View repository job log [here](https://developer.mend.io/github/arthurgeek/vaultwarden-fly-template). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNi41LjMiLCJ1cGRhdGVkSW5WZXIiOiIzNi41LjMiLCJ0YXJnZXRCcmFuY2giOiJtYWluIn0=--> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
As per dani-garcia/vaultwarden#3404, we no longer need a dedicated websocket port.
Fix incorrect tag Do not mount timezone files Rocky Linux 9 does not have an `/etc/timezone` file, which begs the question: why even do this? Upgrade (1.27.0 -> 1.28.0) Fixes mother-of-all-self-hosting/mash-playbook#25 Switch from docker.io to ghcr.io Upgrade (1.28.0 -> 1.28.1) and drop NET_BIND_SERVICE workaround Related to: - mother-of-all-self-hosting/mash-playbook#25 - dani-garcia/vaultwarden#3387 - dani-garcia/vaultwarden#3403 Upgrade (1.28.1 -> 1.29.0) and remove dedicated WebSocket port As per dani-garcia/vaultwarden#3404, we no longer need a dedicated websocket port. Upgrade (1.29.0 -> 1.29.1) Add missing Project source code URL annotation Upgrade (1.29.1 -> 1.29.2) Split vaultwarden_container_additional_networks into vaultwarden_container_additional_networks_auto and vaultwarden_container_additional_networks_custom Add a variable dedicated to the --hostname parameter of the service unit file, default value is vaultwarden_hostname Simplify labels Upgrade (1.29.2 -> 1.30.0) Upgrade (1.30.0 -> 1.30.1) Stop the container gracefully, instead of outright killing it Add vaultwarden_systemd_wanted_systemd_services_list and split required services list into multiple vars Upgrade (1.30.1 -> 1.30.2) Upgrade (1.30.2 -> 1.30.3) copied to own repository
This PR implements a (not yet fully released) new feature of Rocket which allows WebSockets/Upgrade connections.
No more need for multiple ports to be opened for Vaultwarden.
No explicit need for a reverse proxy to get WebSockets to work (Although I still suggest to use a reverse proxy).
rocket_ws
is not yet released.Fixes #685
Fixes #2917
Fixes #1424