Skip to content

Prevent 404 requests to favicon.ico #183

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from
Open

Conversation

slrslr
Copy link

@slrslr slrslr commented Mar 26, 2023

Prevents failed request for favicon.ico resulting in 404 Not found error in server log lines (causing disk write and used space) and web browser requests.

Based on https://stackoverflow.com/a/38917888

Prevents 404 favicon.ico server log lines and web browser requests
@jagerman
Copy link
Member

Can someone confirm whether this works for Safari? (There are comments in the referenced link that it does, but they are also quite old comments).

@slrslr
Copy link
Author

slrslr commented Apr 22, 2023

Can someone confirm whether this works for Safari?

I have asked a Safari user if he sees any warnings or errors on developer console/network tab and he said:
safari

@ghost
Copy link

ghost commented Apr 22, 2023

I tested Safari Version 16.0 (17614.1.25.9.10, 17614) on macOS, but the <head><link rel="icon" href="data:,"><head/> solution didn't work for me.

Note that browsers cache favicon.ico, so you need to make sure to clear the cache every time before testing:

https://oit.colorado.edu/tutorial/clear-web-browser-cache-safari

@Marky-N
Copy link

Marky-N commented May 4, 2023

Screenshot 2023-05-04 at 22 00 19

I was requested to test on MacOS and Safari, here's the result from the console.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants