Skip to content

[24.05] vaultwarden: 1.30.5 -> 1.31.0 -> 1.32.0#334099

Merged
dotlambda merged 5 commits intoNixOS:release-24.05from
SuperSandro2000:vaultwarden-stable
Aug 13, 2024
Merged

[24.05] vaultwarden: 1.30.5 -> 1.31.0 -> 1.32.0#334099
dotlambda merged 5 commits intoNixOS:release-24.05from
SuperSandro2000:vaultwarden-stable

Conversation

@SuperSandro2000
Copy link
Member

@SuperSandro2000 SuperSandro2000 commented Aug 12, 2024

Description of changes

https://github.com/dani-garcia/vaultwarden/releases/tag/1.32.0

Security Fixes
This release has several CVE Reports fixed and we recommend everybody to update to the latest version as soon as possible.

CVE-2024-39924 Fixed via dani-garcia/vaultwarden#4715
CVE-2024-39925 Fixed via dani-garcia/vaultwarden#4837
CVE-2024-39926 Fixed via dani-garcia/vaultwarden#4737

1.31.0 contains breaking changes https://github.com/dani-garcia/vaultwarden/releases/tag/1.31.0 . I am not going to attempt to backport any changes because of the size of the patches.

Things done

  • Built on platform(s)
    • x86_64-linux
    • aarch64-linux
    • x86_64-darwin
    • aarch64-darwin
  • For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)
    • sandbox = relaxed
    • sandbox = true
  • Tested, as applicable:
  • Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
  • Tested basic functionality of all binary files (usually in ./result/bin/)
  • 24.11 Release Notes (or backporting 23.11 and 24.05 Release notes)
    • (Package updates) Added a release notes entry if the change is major or breaking
    • (Module updates) Added a release notes entry if the change is significant
    • (Module addition) Added a release notes entry if adding a new NixOS module
  • Fits CONTRIBUTING.md.

Add a 👍 reaction to pull requests you find important.

@SuperSandro2000 SuperSandro2000 added the 1.severity: security Issues which raise a security issue, or PRs that fix one label Aug 12, 2024
@SuperSandro2000 SuperSandro2000 marked this pull request as draft August 12, 2024 10:08
@dotlambda dotlambda changed the title vaultwarden: 1.30.5 -> 1.31.0 -> 1.32.0 [24.05] vaultwarden: 1.30.5 -> 1.31.0 -> 1.32.0 Aug 12, 2024
@dotlambda
Copy link
Member

We don't want to backport the webvault bumps?

@SuperSandro2000
Copy link
Member Author

yeah, we do. I am currently trying to get a newer rust

@github-actions github-actions bot added the 6.topic: rust General-purpose programming language emphasizing performance, type safety, and concurrency. label Aug 12, 2024
@dotlambda
Copy link
Member

I didn't realize we need a newer Rust. In that case we might just want to apply the security fixes as patches.

Also, please use git cherry-pick -x.

@SuperSandro2000
Copy link
Member Author

SuperSandro2000 commented Aug 12, 2024

I didn't realize we need a newer Rust. In that case we might just want to apply the security fixes as patches.

As already mentioned, I am not going to invest any time into such activities.

Also, please use git cherry-pick -x.

To late now. Also the information in there would be incorrect anyway, since I cherry-picked from my forked branches.


@ofborg build vaultwarden

@ofborg ofborg bot requested a review from dotlambda August 12, 2024 10:38
@ofborg ofborg bot added 11.by: package-maintainer This PR was created by a maintainer of all the package it changes. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. labels Aug 12, 2024
@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/several-information-leaks-in-vaultwarden-1-32-0/50500/1

@dotlambda
Copy link
Member

To late now. Also the information in there would be incorrect anyway, since I cherry-picked from my forked branches.

Just cherry-pick from master instead. If you want, I can open my own PR.

wahjava and others added 4 commits August 12, 2024 13:06
@SuperSandro2000 SuperSandro2000 marked this pull request as ready for review August 12, 2024 11:07
@SuperSandro2000
Copy link
Member Author

Just cherry-pick from master instead. If you want, I can open my own PR.

yeah, whatever. Just did it.

@dotlambda
Copy link
Member

Thank you!

@dotlambda
Copy link
Member

@ofborg test vaultwarden

@SuperSandro2000
Copy link
Member Author

Does it make sense to add release note entries at this point?

@dotlambda
Copy link
Member

Does it make sense to add release note entries at this point?

I doubt it.

@SuperSandro2000
Copy link
Member Author

I am going ahead and will be merging this because of the security situation. I personally don't have the time or knowledge to properly backport the patches and no one spoke up in the last day, so I don't see a quick alternative.

@dotlambda dotlambda merged commit 982eccd into NixOS:release-24.05 Aug 13, 2024
@dotlambda
Copy link
Member

The only reason I hadn't merged yet was to give people a chance to look at the Rust stuff. But they can complain later if they don't like it.

@SuperSandro2000 SuperSandro2000 deleted the vaultwarden-stable branch August 13, 2024 09:06
@SuperSandro2000
Copy link
Member Author

Just for reference: I based it on #298206 so I hope it is fine 😅

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1.severity: security Issues which raise a security issue, or PRs that fix one 6.topic: rust General-purpose programming language emphasizing performance, type safety, and concurrency. 10.rebuild-darwin: 1-10 This PR causes between 1 and 10 packages to rebuild on Darwin. 10.rebuild-linux: 1-10 This PR causes between 1 and 10 packages to rebuild on Linux. 11.by: package-maintainer This PR was created by a maintainer of all the package it changes.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants