[PR #3059] [MERGED] Validate YUBICO_SERVER string (#3003) #3211

New issue

Closed

opened 2026-03-03 09:41:55 +03:00 by kerem · 0 comments

kerem commented

2026-03-03 09:41:55 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/3059
Author: @BlackDex
Created: 12/28/2022
Status: ✅ Merged
Merged: 1/9/2023
Merged by: @dani-garcia

Base: main ← Head: issue-3003

📝 Commits (1)

ca417d3 Validate YUBICO_SERVER string (#3003)

📊 Changes

1 file changed (+11 additions, -2 deletions)

View changed files

📝 src/config.rs (+11 -2)

📄 Description

If the YUBICO_SERVER is defined to an empty string, the whole yubikey implementation doesn't work anymore.

This PR adds a check for this variable that it at least starts with https://.

Resolves #3003

_{🔄 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/3059 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 12/28/2022 **Status:** ✅ Merged **Merged:** 1/9/2023 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `main` ← **Head:** `issue-3003` --- ### 📝 Commits (1) - [`ca417d3`](https://github.com/dani-garcia/vaultwarden/commit/ca417d32578c3b6224c5aa8df56eb776712941b7) Validate YUBICO_SERVER string (#3003) ### 📊 Changes **1 file changed** (+11 additions, -2 deletions) <details> <summary>View changed files</summary> 📝 `src/config.rs` (+11 -2) </details> ### 📄 Description If the `YUBICO_SERVER` is defined to an empty string, the whole yubikey implementation doesn't work anymore. This PR adds a check for this variable that it at least starts with `https://`. Resolves #3003 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>