[PR #4869] [MERGED] Fix Vaultwarden Admin page error messages #3517

New issue

Closed

opened 2026-03-03 09:59:18 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:59:18 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/4869
Author: @BlackDex
Created: 8/18/2024
Status: ✅ Merged
Merged: 8/18/2024
Merged by: @dani-garcia

Base: main ← Head: fix-admin-error-messages

📝 Commits (1)

e190648 Fix Vaultwarden Admin page error messages

📊 Changes

2 files changed (+10 additions, -4 deletions)

View changed files

📝 src/api/admin.rs (+8 -2)
📝 src/static/scripts/admin.js (+2 -2)

📄 Description

Since the change to camelCase variables the error messages in the Vaultwarden Admin were not shown correctly anymore.

This PR fixes this by changing the case of the json key's. Also updated the save and delete of the config to provide a more descriptive error instead of only Io or which ever other error might occure.

Fixes #4834

_{🔄 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/4869 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 8/18/2024 **Status:** ✅ Merged **Merged:** 8/18/2024 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `fix-admin-error-messages` --- ### 📝 Commits (1) - [`e190648`](https://github.com/dani-garcia/vaultwarden/commit/e190648f029a0add07732e3420ebff93df40edfb) Fix Vaultwarden Admin page error messages ### 📊 Changes **2 files changed** (+10 additions, -4 deletions) <details> <summary>View changed files</summary> 📝 `src/api/admin.rs` (+8 -2) 📝 `src/static/scripts/admin.js` (+2 -2) </details> ### 📄 Description Since the change to camelCase variables the error messages in the Vaultwarden Admin were not shown correctly anymore. This PR fixes this by changing the case of the json key's. Also updated the save and delete of the config to provide a more descriptive error instead of only `Io` or which ever other error might occure. Fixes #4834 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>