Re: [Zope-dev] Bug in zc.resourcelibrary?

2007-12-04 Thread Fred Drake
On Dec 4, 2007 9:15 AM, Benji York <[EMAIL PROTECTED]> wrote: > Also, does case matter here? Perhaps a .lower() should be added as well. The major/minor type and the parameter names should be treated as case insensitive. There's a content-type parser in zope.publisher.contentype that should hand

Re: [Zope-dev] Bug in zc.resourcelibrary?

2007-12-04 Thread Benji York
Thomas Lotze wrote: Wouldn't a check like if content_type.split(';', 1)[0] in ('text/html', 'text/xml'): ... be more appropriate? Probably. Remember, zc.resourcelibrary's HTML injection code is a hack -- a necessary hack, but a hack none the less. I can say that because I did it.

[Zope-dev] Bug in zc.resourcelibrary?

2007-12-04 Thread Thomas Lotze
I found some behaviour of zc.resourcelib which I'm not sure is intended that way: The lib takes care to apply only to HTML and XML content, so it tests whether the content type (if set) is among text/html and text/xml: if content_type == 'text/html' or content_type == 'text/xml': ... Howe