Also I think this show go up to var entry = 

There is an early return we are missing here, so entries without a favicon dont get cached (although they dont necessarily need to be cached)