Skip to content

Conversation

x0000ff
Copy link
Contributor

@x0000ff x0000ff commented Oct 14, 2025

The OPDS 1.x is based on Atom, and Atom’s Feed must contain an id tag. I found it useful to parse the <id> tag.

1.1. Introduction
The Open Publication Distribution System (OPDS) Catalog format is a syndication format for electronic publications based on Atom [RFC4287] and HTTP [RFC2616].

atom:feed elements MUST contain exactly one atom:id element.

I left it as optional to avoid breaking compatibility with existing clients.

  • ✅ Tests are updated

@x0000ff x0000ff changed the title Parse the feed identifier for OPDS 1.x Implement parsing of the <id> element in OPDS 1.x feeds Oct 14, 2025
@x0000ff x0000ff force-pushed the feature/parse-feed-id-for-opds-1.0 branch 2 times, most recently from 9e2b993 to 2b15c8c Compare October 14, 2025 20:03
@x0000ff x0000ff force-pushed the feature/parse-feed-id-for-opds-1.0 branch from 2b15c8c to 07f3a99 Compare October 21, 2025 13:36
Copy link
Member

@mickael-menu mickael-menu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looking good, thank you. Please note that we will revamp the OPDS models next year, and this kind of information will be available as part of the extension models for metadata instead of a native property.

@mickael-menu mickael-menu merged commit 8875768 into readium:develop Oct 21, 2025
5 checks passed
@x0000ff x0000ff deleted the feature/parse-feed-id-for-opds-1.0 branch October 21, 2025 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants