Do not indefinitely retain failed favicon lookups

[jfroelich]

Do not keep favicon lookup failure entries around forever, The failure guards should eventually let the request succeed somehow. Probably failure entries should also have an expiration period. But based on expires date set in entry instead of externally, so that I can have different expiration dates for non-failure cache items and failure cache items. The compact function could also check for these and remove expired