Copy

Defect #49

text/plain should not be an allowed MIME type

Added by Geoffrey Sneddon 863 days ago. Updated 620 days ago.

Status:Verified Start:2008-03-17
Priority:High Due date:
Assigned to:Geoffrey Sneddon % Done:

0%
Category:HTTP
Target version:1.1.2
Affected Version:

1.1.1

 PHP Version:

5.2.4
mbstring enabled:

Yes

 iconv enabled:

Yes
cURL enabled:

Yes

 zlib enabled:

Yes

Description

text/plain was added while Ryan was doing debugging and should have never been commited, yet alone ever released. application/rdf+xml is allowable for RSS 0.90 and RSS 1.0 (and some other RSS feeds are mis-served as it, such as http://nick.typepad.com/blog/index.rss)

Associated revisions

Revision 966
Added by Geoffrey Sneddon 863 days ago

Allow application/rdf+xml and disallow text/plain. Fixes #49.

Revision 967
Added by Geoffrey Sneddon 863 days ago

Duplicating application/rdf+xml is the first sign of madness. Refs #49.

History

Updated by Geoffrey Sneddon 863 days ago

  • Status changed from New to Fixed

Applied in changeset r966.

Updated by Ryan Parman 863 days ago

Looking at the commits, it appears that application/rdf+xml is in the "allow" list twice. Is this correct?

http://bugs.simplepie.org/repositories/diff/sp1/trunk/simplepie.inc?rev=966
http://bugs.simplepie.org/repositories/diff/sp1/branches/1.1/simplepie.inc?rev=966

Updated by Geoffrey Sneddon 863 days ago

n00b'd. See r967.

Updated by Ryan Parman 863 days ago

So... how is this different from the 1.1.1 release other than disallowing text/plain? Is it really still a high priority, or can this now be pushed off until there are more bugs to fix?

Updated by Geoffrey Sneddon 853 days ago

I'm moving 1.1.2 off to 2nd June (the "thank God Geoffrey's exams are over" release), and we can just ship whatever we have then, as I doubt we'll ship 1.2 before August at the earliest.

Updated by Geoffrey Sneddon 627 days ago

  • Status changed from Fixed to Verified

Updated by Geoffrey Sneddon 620 days ago

  • Subject changed from text/plain allowed MIME type, application/rdf+xml not to text/plain allowed MIME type

Updated by Geoffrey Sneddon 620 days ago

  • Subject changed from text/plain allowed MIME type to text/plain should not be an allowed MIME type

Also available in: Atom PDF