iTunes extension not working while the extensions map does contain the respective information about rss HOT 8 CLOSED

So, I reviewed the code and it seems that I overestimated the complexity. It looks a bit easier now #159.

from rss.

Extensions are recognized by a namespace URI. iTunes URI in your example is http://www.itunes.com/DTDs/Podcast-1.0.dtd while the crate looks for http://www.itunes.com/dtds/podcast-1.0.dtd.

from rss.

Wouldn't a toLowercase comparison solve the issue?

from rss.

If you'd like to I can add a pull request for this. @andy128k

from rss.

Tough question. Formally URIs are case sensitive and such a change may be risky.

from rss.

Good point. The question is will someone else create a schema that is equal to the iTunes url but change a character in the url to lower/uppercase. Probably not. So I think the chance of breaking an existing project using your library is very minimal.

from rss.

https://help.apple.com/itc/podcasts_connect/#/itcb54353390

Without this declaration, Podcasts Connect ignores all tags specific to Apple Podcasts. The namespace definition is case sensitive, and must be entered as shown.

It's not about someone creating a schema (that's unlikely and that should be someone with access to www.itunes.com domain).

But someone may create a feed with multiple namespaces:

<rss xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd" xmlns:ITUNES="http://www.itunes.com/DTDs/Podcast-1.0.dtd" version="2.0">
   <channel>
       <itunes:author>Author 1</itunes:author>
       <ITUNES:author>Author 2</ITUNES:author>
   </channel>
</rss>

Now we have an issue with merging formally two extensions into a single one. OTOH someone may create an alias like <rss xmlns:itunes="http://www.itunes.com/dtds/podcast-1.0.dtd" xmlns:ITUNES="http://www.itunes.com/dtds/podcast-1.0.dtd" version="2.0"> and we still have this problem.

I'd say, this is much bigger issue than just add toLowercase.

from rss.

Thank you for the changes 😃.

from rss.