Skip to content

Prevent 404 requests to favicon.ico - #183

Open
slrslr wants to merge 1 commit into
oxen-io:devfrom
slrslr:patch-1
Open

slrslr wants to merge 1 commit into
oxen-io:devfrom
slrslr:patch-1

Conversation

@slrslr

@slrslr slrslr commented Mar 26, 2023

Copy link
Copy Markdown

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
Copy Markdown
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

slrslr commented Apr 22, 2023

Copy link
Copy Markdown
Author

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

ghost commented Apr 22, 2023 •

Copy link
Copy Markdown

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

Marky-N commented May 4, 2023

Copy link
Copy Markdown

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