Skip to content
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

feat: Support for storing passkeys in the vault #3593

Merged

Conversation

GeekCornerGH
Copy link
Contributor

@GeekCornerGH GeekCornerGH commented Jun 19, 2023

Implements the ability to store passkeys in the vault

How to test

  1. Checkout this branch, then install dependencies with npm i (npm ci didn't worked, where npm i somehow worked in my case)
  2. Enable feature flag for the extension. As I didn't figured out how to do this properly, I changed the values there and there to make them always return true
  3. Follow the docs to build and install the modified extension in your browser (I installed it in a custom profile)
  4. Go to a website supporting passkeys. I used passkeys.io, and also tested against webauthn.io but any website supporting webauthn should actually work, then register your key

Demo

0t3iC11yBS.mp4

Notes

Upstream PR: bitwarden/server#2679
Tested on Brave 1.54.65 Chromium: 114.0.5735.133 (Official build) nightly (64 bits)
The feature isn't released yet, but there shouldn't be any big changes as Vaultwarden doesn't type the properties of ciphers the way upstream server does

@GeekCornerGH GeekCornerGH force-pushed the feature/store-passkeys-in-the-vault branch from eb7dcb0 to 1074315 Compare June 22, 2023 16:49
@GeekCornerGH GeekCornerGH requested a review from dani-garcia June 22, 2023 16:49
@dani-garcia dani-garcia merged commit e7f083d into dani-garcia:main Jun 22, 2023
arthurgeek referenced this pull request in arthurgeek/vaultwarden-fly-template Jul 12, 2023
[![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
[#&#8203;3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304)
thanks [@&#8203;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:
[#&#8203;3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593),
thanks [@&#8203;GeekCornerGH](https://togithub.com/GeekCornerGH)!

#### What's Changed

- check if reset password policy is enabled by
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;BlackDex](https://togithub.com/BlackDex) in
[https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659)

#### New Contributors

- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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>
arthurgeek referenced this pull request in arthurgeek/vaultwarden-fly Jul 12, 2023
[![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
[#&#8203;3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304)
thanks [@&#8203;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:
[#&#8203;3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593),
thanks [@&#8203;GeekCornerGH](https://togithub.com/GeekCornerGH)!

#### What's Changed

- check if reset password policy is enabled by
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;BlackDex](https://togithub.com/BlackDex) in
[https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659)

#### New Contributors

- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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>
arthurgeek referenced this pull request in arthurgeek/vaultwarden-fly Jul 12, 2023
[![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
[#&#8203;3304](https://togithub.com/dani-garcia/vaultwarden/issues/3304)
thanks [@&#8203;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:
[#&#8203;3593](https://togithub.com/dani-garcia/vaultwarden/issues/3593),
thanks [@&#8203;GeekCornerGH](https://togithub.com/GeekCornerGH)!

#### What's Changed

- check if reset password policy is enabled by
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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 [@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;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
[@&#8203;BlackDex](https://togithub.com/BlackDex) in
[https://github.com/dani-garcia/vaultwarden/pull/3659](https://togithub.com/dani-garcia/vaultwarden/pull/3659)

#### New Contributors

- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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)
- [@&#8203;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>
@meowcoder
Copy link

  1. Enable feature flag for the extension. As I didn't figured out how to do this properly

The extension queries the /api/config endpoint for this.
One can enable passkeys like this:

diff --git a/src/api/core/mod.rs b/src/api/core/mod.rs
index f142468..2e50870 100644
--- a/src/api/core/mod.rs
+++ b/src/api/core/mod.rs
@@ -200,6 +200,9 @@ fn config() -> Json<Value> {
           "name": "Vaultwarden",
           "url": "https://github.com/dani-garcia/vaultwarden"
         },
+        "featureStates": {
+            "fido2-vault-credentials": true
+        },
         "environment": {
           "vault": domain,
           "api": format!("{domain}/api"),

@stefan0xC
Copy link
Contributor

stefan0xC commented Oct 9, 2023

@meowcoder I don't think that's enough because this feature has not even been merged yet upstream. See bitwarden/server#2679 and bitwarden/clients#4715

And because Bitwarden has changed their passkey implementation from it's own item type to a login item field (c.f. bitwarden/server#3261) we also have to update our implementation as well.

In general, I don't think that we should enable features before they are stable.

@ItsShadowCone
Copy link

Upstream support has been released, so should be stable now: see #3355

What's necessary to get there for vaultwarden too?

@agelwarg
Copy link

agelwarg commented Nov 2, 2023

Upstream support has been released, so should be stable now: see #3355

What's necessary to get there for vaultwarden too?

Also looks like the updates to the clients / browser extensions have been rolled out as well (just updated on my iPhone) and it works (against upstream, not Vaultwarden).

@GeekCornerGH
Copy link
Contributor Author

GeekCornerGH commented Nov 3, 2023

Working on a new PR

@adriy-be
Copy link

adriy-be commented Nov 4, 2023

I retried today and that work now. Thanks for your work ! 😁

Hi, I have updated my docker file to testing tag. But It's not working on brave.
There is diagnostics of vaultwarden:
Server Installed: 1.29.2-93da5091
Server Latest: 1.29.2-93da5091
Web Installed: 2023.10.0

And my extension:
version : 2023.10.0
Server version: 1.29.2

Brave version:
[Version 1.60.110 Chromium: 119.0.6045.105 (Official Build) (64-bit)]

The message I have got when I try:
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants