Because that's the particular use case we saw in our logs resulting in uncaught exceptions :)
IMO your examples call for changing the blacklist to whitelist as discussed. See any noteworthy issues w/ that option? Thanks, John On Thu, Feb 25, 2010 at 6:19 PM, <[email protected]> wrote: > Why was image/x-icon added to the blacklist, but not: > image/vnd.microsoft.icon, image/ico, image/icon, text/ico, > application/ico > > > On 2010/02/25 23:39:39, zhoresh wrote: > >> This is the simple fix of just adding type to blacklist. >> > > Is it time to revisit this and run the encoding detector only on >> > text/html or > >> text/* instead of using blacklist? >> > > > > http://codereview.appspot.com/224064/show >
