Re: [dev] [surf] [patch] Don't load favicon.ico

From: Christoph Lohmann <>
Date: Thu, 06 Dec 2012 13:34:04 +0100


On Thu, 06 Dec 2012 13:34:04 +0100 Nick <> wrote:
> On Thu, Dec 06, 2012 at 07:58:19AM +0100, Christoph Lohmann wrote:
> > I can’t apply this patch. It’s living on the assumption that favicons
> > will end in »favicon.ico«. Really blocking the favicon would require
> > to use the situation of webkit, where it really does that blind request
> > for the favicon.ico. Your solution will block every URI ending in
> > »favicon.ico«.
> Well webkit is making the assumption that there may be a
> favicon.ico, and requesting it. Future versions may think better of
> this, but for now they aren't, and I think overriding this
> particularly stupid behaviour is fine.

The patch is now in surf, but it is searching for »/favicon.ico«. This
should work too.

But this patch adds the hook that would allow easy adblocking. :D What’s
just needed is a stripped down regexp list that can be compiled into
surf. If the benchmark is fast enough it should be good to go.


Christoph Lohmann
Received on Thu Dec 06 2012 - 13:34:04 CET

This archive was generated by hypermail 2.3.0 : Thu Dec 06 2012 - 13:48:04 CET