[PR #1584] [MERGED] Some admin interface updates. #3406

Closed
opened 2025-10-09 18:22:50 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/1584
Author: @BlackDex
Created: 4/4/2021
Status: Merged
Merged: 4/6/2021
Merged by: @dani-garcia

Base: masterHead: admin-interface


📝 Commits (1)

  • 95fc88a Some admin interface updates.

📊 Changes

10 files changed (+50 additions, -56 deletions)

View changed files

📝 docker/Dockerfile.j2 (+0 -4)
📝 docker/amd64/Dockerfile (+0 -1)
📝 docker/amd64/Dockerfile.alpine (+0 -1)
📝 docker/arm64/Dockerfile (+0 -1)
📝 docker/armv6/Dockerfile (+0 -1)
📝 docker/armv7/Dockerfile (+0 -1)
📝 docker/armv7/Dockerfile.alpine (+0 -1)
📝 src/api/admin.rs (+18 -9)
📝 src/db/mod.rs (+20 -36)
📝 src/static/templates/admin/diagnostics.hbs (+12 -1)

📄 Description

  • Fixed bug when web-vault is disabled.
  • Updated sql-server version check to be simpler thx to @weiznich ( https://github.com/dani-garcia/bitwarden_rs/pull/1548#discussion_r604767196 )
  • Use VACUUM INTO to create a SQLite backup instead of using the external sqlite3 application.
    • This also removes the dependancy of having the sqlite3 packages installed on the final image unnecessary, and thus removed it.
  • Updated backup filename to also have the current time.
  • Add specific bitwarden_rs web-vault version check (to match letter patched versions)
    Will work when https://github.com/dani-garcia/bw_web_builds/pull/33 is build (But still works without it also).

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/dani-garcia/vaultwarden/pull/1584 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 4/4/2021 **Status:** ✅ Merged **Merged:** 4/6/2021 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `admin-interface` --- ### 📝 Commits (1) - [`95fc88a`](https://github.com/dani-garcia/vaultwarden/commit/95fc88ae5bef5f4d1e9a8da4f5de7c955fb75a19) Some admin interface updates. ### 📊 Changes **10 files changed** (+50 additions, -56 deletions) <details> <summary>View changed files</summary> 📝 `docker/Dockerfile.j2` (+0 -4) 📝 `docker/amd64/Dockerfile` (+0 -1) 📝 `docker/amd64/Dockerfile.alpine` (+0 -1) 📝 `docker/arm64/Dockerfile` (+0 -1) 📝 `docker/armv6/Dockerfile` (+0 -1) 📝 `docker/armv7/Dockerfile` (+0 -1) 📝 `docker/armv7/Dockerfile.alpine` (+0 -1) 📝 `src/api/admin.rs` (+18 -9) 📝 `src/db/mod.rs` (+20 -36) 📝 `src/static/templates/admin/diagnostics.hbs` (+12 -1) </details> ### 📄 Description - Fixed bug when web-vault is disabled. - Updated sql-server version check to be simpler thx to @weiznich ( https://github.com/dani-garcia/bitwarden_rs/pull/1548#discussion_r604767196 ) - Use `VACUUM INTO` to create a SQLite backup instead of using the external sqlite3 application. - This also removes the dependancy of having the sqlite3 packages installed on the final image unnecessary, and thus removed it. - Updated backup filename to also have the current time. - Add specific bitwarden_rs web-vault version check (to match letter patched versions) Will work when https://github.com/dani-garcia/bw_web_builds/pull/33 is build (But still works without it also). --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2025-10-09 18:22:50 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/vaultwarden#3406