[PR #876] [MERGED] Make panics logable (as warn) #2812

New issue

Closed

opened 2026-03-03 08:53:12 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 08:53:12 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/876
Author: @BlackDex
Created: 2/25/2020
Status: ✅ Merged
Merged: 3/16/2020
Merged by: @dani-garcia

Base: master ← Head: log-panics

📝 Commits (3)

7439aeb Make panics logable (as warn)
bcbe617 Merge branch 'master' of https://github.com/dani-garcia/bitwarden_rs into log-panics
bd09fe1 Updated code so backtraces are logged also.

📊 Changes

3 files changed (+55 additions, -0 deletions)

View changed files

📝 Cargo.lock (+1 -0)
📝 Cargo.toml (+3 -0)
📝 src/main.rs (+51 -0)

📄 Description

panic!()'s only appear on stderr, this makes tracking down some strange
issues harder with the usage of docker since stderr does not get logged
into the bitwarden.log file. This change logs the message to stdout and
the logfile when activated.

_{🔄 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/876 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 2/25/2020 **Status:** ✅ Merged **Merged:** 3/16/2020 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `log-panics` --- ### 📝 Commits (3) - [`7439aeb`](https://github.com/dani-garcia/vaultwarden/commit/7439aeb63ef692510c121a0e9f090d58de8d87e2) Make panics logable (as warn) - [`bcbe617`](https://github.com/dani-garcia/vaultwarden/commit/bcbe6177b8a0cf7d320a683b1c636aec363a0d0f) Merge branch 'master' of https://github.com/dani-garcia/bitwarden_rs into log-panics - [`bd09fe1`](https://github.com/dani-garcia/vaultwarden/commit/bd09fe1a3d9b7f31644fb5fb9f9c38a30c893dac) Updated code so backtraces are logged also. ### 📊 Changes **3 files changed** (+55 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+1 -0) 📝 `Cargo.toml` (+3 -0) 📝 `src/main.rs` (+51 -0) </details> ### 📄 Description panic!()'s only appear on stderr, this makes tracking down some strange issues harder with the usage of docker since stderr does not get logged into the bitwarden.log file. This change logs the message to stdout and the logfile when activated. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>