Joey Hess <[email protected]> writes:

> We don't know tidy isn't installed, it might have failed to tidy the
> html (can happen for very screwy html), or therotectically, have
> crashed. So returning the untidied html is not right -- the site could
> be relying on htmltidy to eg, close all unclosed tags in aggregated
> content.

I think that throwing away aggregated content if tidy couldn't cope with
it is not right either. Such behavior annoys users a great deal and it
is likely to confuse them because the "bug" seems unreproducible due to
varying external inputs over which the user has no control.

> Printing a warning message seems reasonable, although it might be
> annoying to have ikiwiki cron jobs complaining every time aggregated
> content could not be tidied.

How about a warning that can the user can opt out of by setting a
configuration variable?

-Hilko



-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to