caddyserver / caddy

Fast and extensible multi-platform HTTP/1-2-3 web server with automatic HTTPS
https://caddyserver.com
Apache License 2.0
55.45k stars 3.91k forks source link

[file_server browse] missing favicon #6421

Closed trmdi closed 4 days ago

trmdi commented 4 days ago

For example, if you use duckdns and have visited www.duckdns.org, after that when you go to file.example.duckdns.org the duck icon is shown like this: image

I think we need to add the favicon tag.

mohammed90 commented 4 days ago

We can't assume the favicon icon for the users, and embedding the Caddy icon is excessive. Unlike js/css libs, pointing at Caddy's favicon on our website would also be excessive and violates privacy of the users. You can set it yourself in 2 ways:

mholt commented 3 days ago

We could embed the Caddy icon (I'm not personally opposed to that), but I'm not sure people would want our icon shown on their site. It might confuse users when their tab shows a different icon than what belongs to the site they're visiting.

The more I think about it the more I think the favicon is working as expected.