[PR #6453] [MERGED] Use an older version of mariadb to prevent a panic #24646

Closed
opened 2026-05-22 02:11:18 -05:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/6453
Author: @BlackDex
Created: 11/10/2025
Status: Merged
Merged: 11/10/2025
Merged by: @dani-garcia

Base: mainHead: use-older-mariadb-lib


📝 Commits (3)

  • e1131c8 Use an older version of mariadb to prevent a panic
  • e3c871d Update GHA versions
  • a4b1771 Resolve docker build check issue

📊 Changes

5 files changed (+99 additions, -67 deletions)

View changed files

📝 .github/workflows/release.yml (+1 -1)
📝 Cargo.lock (+60 -60)
📝 Cargo.toml (+4 -4)
📝 docker/Dockerfile.debian (+17 -1)
📝 docker/Dockerfile.j2 (+17 -1)

📄 Description

The Debian builds use a newer version of libmariadb which causes Diesel to panic on certain queries. This commit prevents this by using an older version of libmariadb which doesn't cause this panic.

The Alpine based versions use a patched version which reverts the commit in the libmariadb library which causes this panic. In the future this might be fixed in Diesel it self (https://github.com/dani-garcia/vaultwarden/issues/6416#issuecomment-3508822097), but until then, we use an older version of the library.

Fixes #6416


🔄 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/6453 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 11/10/2025 **Status:** ✅ Merged **Merged:** 11/10/2025 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `use-older-mariadb-lib` --- ### 📝 Commits (3) - [`e1131c8`](https://github.com/dani-garcia/vaultwarden/commit/e1131c8fa0df4863d43f5af82c657499528c689c) Use an older version of mariadb to prevent a panic - [`e3c871d`](https://github.com/dani-garcia/vaultwarden/commit/e3c871d6bb19209a3a7548f5d677ce95a4e8e590) Update GHA versions - [`a4b1771`](https://github.com/dani-garcia/vaultwarden/commit/a4b1771fd786b601da5bcdcd0e85a17d631471cf) Resolve docker build check issue ### 📊 Changes **5 files changed** (+99 additions, -67 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/release.yml` (+1 -1) 📝 `Cargo.lock` (+60 -60) 📝 `Cargo.toml` (+4 -4) 📝 `docker/Dockerfile.debian` (+17 -1) 📝 `docker/Dockerfile.j2` (+17 -1) </details> ### 📄 Description The Debian builds use a newer version of libmariadb which causes Diesel to panic on certain queries. This commit prevents this by using an older version of libmariadb which doesn't cause this panic. The Alpine based versions use a patched version which reverts the commit in the libmariadb library which causes this panic. In the future this might be fixed in Diesel it self (https://github.com/dani-garcia/vaultwarden/issues/6416#issuecomment-3508822097), but until then, we use an older version of the library. Fixes #6416 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
GiteaMirror added the pull-request label 2026-05-22 02:11:18 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#24646