search

Podcastindex-org / podcast-namespace

A wholistic rss namespace for podcasting
Creative Commons Zero v1.0 Universal
385 stars 116 forks source link

Podcast chapters #302

Open klaaz0r opened 3 years ago

klaaz0r commented 3 years ago

I am implementing chapters in my app but I am curious what has been the reasoning behind having chapters behind another API call? Why not make the chapters part of the actual feed?

   <podcast:chapters version="2.0">
        <podcast:chapter start="0" title="Welcome" />
        <podcast:chapter start="3:07" title="Star episode"  />
   </podcast:chapters>

(Simple example)

I understand the proposal is already gone trough but was there a specific reason to have chapters in JSON?

PofMagicfingers commented 3 years ago

It was moved to an external file to prevent the feed becoming too big, or cluttered, for performance and bandwith concerns.

Le lun. 11 oct. 2021, 19:16, Klaas Foppen @.***> a écrit :

I am implementing chapters in my app but I am curious what has been the reasoning behind having chapters behind another API call? Why not make the chapters part of the actual feed?

(Simple example) I understand the proposal is already gone trough but was there a specific reason to have chapters in JSON? — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub , or unsubscribe . Triage notifications on the go with GitHub Mobile for iOS or Android .
theDanielJLewis commented 3 years ago

Plus, the chapters are simply one part of additional episode data. So it's really not a "chapters file" but an "episode metadata" file that can contain other data.

PofMagicfingers commented 3 years ago

Yes, we were discussing in #184 about moving all "episode metadata" to a file. Podcast clients would download this file when downloading episode or accessing the episode infos

daveajones commented 3 years ago

I'll add that we were told by more than one large host that adding the chapters to the feed itself would prohibit them from implementing. So, it was just a no-go out of the gate.

tomrossi7 commented 3 years ago

Yeah adding the chapters to the feed is a lot of clutter. Every app doesn't need the chapters for every episode. It makes sense that it is an additional HTTP call. The proper use of ETags and HTTP 304s should eliminate the need for a hash as well.

PofMagicfingers commented 3 years ago

The proper use of ETags and HTTP 304s should eliminate the need for a hash as well

Well, using a hash will eliminate the need of a request at all IMHO 😁 But that's a debate to have in the #184 issue

saerdnaer commented 2 years ago

@klaaz0r If you want to put your chapters directly into the RSS feed you can use https://podlove.org/simple-chapters/ which was established in 2012 and is already supported by Podcast Addict, Antenna Pod, Podcat, etc.

See also https://github.com/Podcastindex-org/podcast-namespace/blob/main/docs/other-recommendations.md#feed-paging-and-archiving-rfc5005

So in my opinion this issue can be closed.