[GH-ISSUE #417] filling the log with /favicon.{ico,png} 404 #307

New issue

Closed

opened 2026-02-26 10:30:43 +03:00 by kerem · 5 comments

kerem commented

2026-02-26 10:30:43 +03:00

Owner

Originally created by @ThomasWaldmann on GitHub (Apr 9, 2019).
Original GitHub issue: https://github.com/nsupdate-info/nsupdate.info/issues/417

seeing multiple log entries from same IP at same time requesting these resulting in a 404.

check if we link to that URL somewhere or if it is implicit like old browsers did.

handle the URL, so no 404 is logged.

Originally created by @ThomasWaldmann on GitHub (Apr 9, 2019). Original GitHub issue: https://github.com/nsupdate-info/nsupdate.info/issues/417 seeing multiple log entries from same IP at same time requesting these resulting in a 404. check if we link to that URL somewhere or if it is implicit like old browsers did. handle the URL, so no 404 is logged.

kerem closed this issue

2026-02-26 10:30:44 +03:00

kerem commented

2026-02-26 10:30:44 +03:00

Author

Owner

@elnappo commented on GitHub (Apr 9, 2019):

Browsers are requesting these files per default, with our new logo we should add a favicon :)

@elnappo commented on GitHub (Apr 9, 2019): Browsers are requesting these files per default, with our new logo we should add a favicon :)

kerem commented

2026-02-26 10:30:44 +03:00

Author

Owner

@ThomasWaldmann commented on GitHub (Apr 9, 2019):

don't we have some head tag that points this to a specific location in the static files?

@ThomasWaldmann commented on GitHub (Apr 9, 2019): don't we have some head tag that points this to a specific location in the static files?

kerem commented

2026-02-26 10:30:44 +03:00

Author

Owner

@ThomasWaldmann commented on GitHub (Apr 9, 2019):

we do:

        <link rel="icon" type="image/png" href="/static/favicon.png" />

@ThomasWaldmann commented on GitHub (Apr 9, 2019): we do: ``` <link rel="icon" type="image/png" href="/static/favicon.png" /> ```

kerem commented

2026-02-26 10:30:44 +03:00

Author

Owner

@elnappo commented on GitHub (Apr 10, 2019):

It seems like not all browsers are respecting this property. I also see /favicon.ico requests in logfiles while providing a link to /favicon.svg. I guess adding a /favicon.ico should "fix" this problem.

@elnappo commented on GitHub (Apr 10, 2019): It seems like not all browsers are respecting this property. I also see `/favicon.ico` requests in logfiles while providing a link to `/favicon.svg`. I guess adding a `/favicon.ico` should "fix" this problem.

kerem commented

2026-02-26 10:30:44 +03:00

Author

Owner

@ThomasWaldmann commented on GitHub (Apr 15, 2019):

did an alias on nginx level, so the requests don't get through to the wsgi app.

@ThomasWaldmann commented on GitHub (Apr 15, 2019): did an alias on nginx level, so the requests don't get through to the wsgi app.

kerem referenced this issue

2026-02-26 10:31:33 +03:00

[PR #307] [MERGED] update CHANGES #454