[PR #638] [MERGED] Add sqlite binary into the docker images #2762

New issue

Closed

opened 2026-03-03 08:52:58 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 08:52:58 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/638
Author: @mprasil
Created: 9/30/2019
Status: ✅ Merged
Merged: 10/1/2019
Merged by: @dani-garcia

Base: master ← Head: add_sqlite_binary

📝 Commits (2)

acdd429 Add sqlite binary into the docker images
d6e9af9 Remove the unnecessary check for sqlite

📊 Changes

8 files changed (+8 additions, -1 deletions)

View changed files

📝 docker/aarch64/sqlite/Dockerfile (+1 -0)
📝 docker/amd64/postgresql/Dockerfile (+1 -0)
📝 docker/amd64/postgresql/Dockerfile.alpine (+1 -0)
📝 docker/amd64/sqlite/Dockerfile (+1 -0)
📝 docker/amd64/sqlite/Dockerfile.alpine (+1 -0)
📝 docker/armv6/sqlite/Dockerfile (+1 -0)
📝 docker/armv7/sqlite/Dockerfile (+1 -0)
📝 src/api/admin.rs (+1 -1)

📄 Description

This is done to enable backup functionality in the admin interface while
we're waiting for the libsqlite-sys 0.17 to bubble up in the upstream
dependencies. Then we can start using VACUUM INTO

This also extends the check for the sqlite binary to also try sqlite3
as this is the name of the binary in baseimage distributions we use.

_{🔄 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/638 **Author:** [@mprasil](https://github.com/mprasil) **Created:** 9/30/2019 **Status:** ✅ Merged **Merged:** 10/1/2019 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `add_sqlite_binary` --- ### 📝 Commits (2) - [`acdd429`](https://github.com/dani-garcia/vaultwarden/commit/acdd42935b7e71461d51ab2bb92610bfec8a6601) Add sqlite binary into the docker images - [`d6e9af9`](https://github.com/dani-garcia/vaultwarden/commit/d6e9af909be8be79bf4f301d25a343df6108efd7) Remove the unnecessary check for sqlite ### 📊 Changes **8 files changed** (+8 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `docker/aarch64/sqlite/Dockerfile` (+1 -0) 📝 `docker/amd64/postgresql/Dockerfile` (+1 -0) 📝 `docker/amd64/postgresql/Dockerfile.alpine` (+1 -0) 📝 `docker/amd64/sqlite/Dockerfile` (+1 -0) 📝 `docker/amd64/sqlite/Dockerfile.alpine` (+1 -0) 📝 `docker/armv6/sqlite/Dockerfile` (+1 -0) 📝 `docker/armv7/sqlite/Dockerfile` (+1 -0) 📝 `src/api/admin.rs` (+1 -1) </details> ### 📄 Description This is done to enable backup functionality in the admin interface while we're waiting for the libsqlite-sys 0.17 to bubble up in the upstream dependencies. Then we can start using `VACUUM INTO` This also extends the check for the sqlite binary to also try `sqlite3` as this is the name of the binary in baseimage distributions we use. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>