[PR #741] [MERGED] Add an option to fetch and parse href="data:image" #6440

Closed
opened 2026-03-07 20:59:05 -06:00 by GiteaMirror · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/dani-garcia/vaultwarden/pull/741
Author: @BlackDex
Created: 11/22/2019
Status: Merged
Merged: 11/22/2019
Merged by: @dani-garcia

Base: masterHead: icon-datauri


📝 Commits (1)

  • b209c1b Add an option to fetch and parse href="data:image"

📊 Changes

3 files changed (+54 additions, -17 deletions)

View changed files

📝 Cargo.lock (+24 -8)
📝 Cargo.toml (+4 -0)
📝 src/api/icons.rs (+26 -9)

📄 Description

Some sites are using base64 encoded inline images for favicons.
This will try to match those with some sane checks and return that.
These icons will have lower prio then the icons with a normal URL.


🔄 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/741 **Author:** [@BlackDex](https://github.com/BlackDex) **Created:** 11/22/2019 **Status:** ✅ Merged **Merged:** 11/22/2019 **Merged by:** [@dani-garcia](https://github.com/dani-garcia) **Base:** `master` ← **Head:** `icon-datauri` --- ### 📝 Commits (1) - [`b209c1b`](https://github.com/dani-garcia/vaultwarden/commit/b209c1bc4d3f31a9b7061ee2b4aef276faaef3f8) Add an option to fetch and parse href="data:image" ### 📊 Changes **3 files changed** (+54 additions, -17 deletions) <details> <summary>View changed files</summary> 📝 `Cargo.lock` (+24 -8) 📝 `Cargo.toml` (+4 -0) 📝 `src/api/icons.rs` (+26 -9) </details> ### 📄 Description Some sites are using base64 encoded inline images for favicons. This will try to match those with some sane checks and return that. These icons will have lower prio then the icons with a normal URL. --- <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-03-07 20:59:05 -06:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/vaultwarden#6440