[PR #2032] [MERGED] Added DbConn to /alive healthcheck #3314

New Issue

OVERLORD · 2025-10-09T18:21:04+03:00

OVERLORD commented

2025-10-09 18:21:04 +03:00

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/2032
Author: @BlackDex
Created: 10/9/2021
Status: ✅ Merged
Merged: 10/18/2021
Merged by: @dani-garcia

Base: main ← Head: alive-db-check

📝 Commits (1)

881524b Added DbConn to /alive healthcheck

📊 Changes

1 file changed (+3 additions, -1 deletions)

View changed files

📝 src/api/web.rs (+3 -1)

📄 Description

During a small discusson on Matrix it seems logical to have the /alive
endpoint also check if the database connection still works.

The reason for this was regarding a certificate which failed/expired
while vaultwarden and the database were still up-and-running, but
suddenly vaultwarden couldn't connect anymore.

With this DbConn added to /alive, it will be more accurate, because
of vaultwarden can't reach the database, it isn't alive.

_{🔄 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/2032 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 10/9/2021 **Status:** ✅ Merged **Merged:** 10/18/2021 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `alive-db-check` --- ### 📝 Commits (1) - [`881524b`](https://github.com/dani-garcia/vaultwarden/commit/881524bd5466c19875403b46915ac2ec5762b0ca) Added DbConn to /alive healthcheck ### 📊 Changes **1 file changed** (+3 additions, -1 deletions) <details> <summary>View changed files</summary> 📝 `src/api/web.rs` (+3 -1) </details> ### 📄 Description During a small discusson on Matrix it seems logical to have the /alive endpoint also check if the database connection still works. The reason for this was regarding a certificate which failed/expired while vaultwarden and the database were still up-and-running, but suddenly vaultwarden couldn't connect anymore. With this `DbConn` added to `/alive`, it will be more accurate, because of vaultwarden can't reach the database, it isn't alive. --- <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:21:04 +03:00

OVERLORD closed this issue

2025-10-09 18:21:06 +03:00

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/vaultwarden#3314