Skip to content

Add additional media types to OPDS Browser#6443

Merged
Frenzie merged 2 commits intokoreader:masterfrom
gotson:patch-1
Jul 28, 2020
Merged

Add additional media types to OPDS Browser#6443
Frenzie merged 2 commits intokoreader:masterfrom
gotson:patch-1

Conversation

@gotson
Copy link
Copy Markdown
Contributor

@gotson gotson commented Jul 28, 2020

The media types for CBZ and CBR (added in #5940) are deprecated, and replaced by vnd.comicbook+zip and vnd.comicbook+rar. In addition, many OPDS servers will use the actual media type of the file, ie application/zip and application/x-rar-compressed.

See here about the new vnd types:


This change is Reviewable

gotson added 2 commits July 28, 2020 21:01
The media types for CBZ and CBR (added in #5940) are deprecated, and replaced by `vnd.comicbook+zip` and `vnd.comicbook+rar`. In addition, many OPDS servers will use the actual media type of the file, ie `application/zip` and `application/x-rar-compressed`.

See here about the new `vnd` types:
- https://www.iana.org/assignments/media-types/application/vnd.comicbook+zip
- https://www.iana.org/assignments/media-types/application/vnd.comicbook-rar
@gotson
Copy link
Copy Markdown
Contributor Author

gotson commented Jul 28, 2020

I also added application/vnd.rar which is replacing application/x-rar-compressed

@Frenzie Frenzie added this to the 2020.08 milestone Jul 28, 2020
@Frenzie Frenzie merged commit 072d960 into koreader:master Jul 28, 2020
@Frenzie
Copy link
Copy Markdown
Member

Frenzie commented Jul 28, 2020

Thanks!

Frenzie pushed a commit that referenced this pull request Jul 29, 2020
following #6443 i thought i would check the other types

application/x-mobi8-ebook is replaced by application/vnd.amazon.mobi8-ebook (see https://www.iana.org/assignments/media-types/application/vnd.amazon.mobi8-ebook)
application/djvu doesn't seem to be an official one. image/x-djvu was the previous one, and is replaced by image/vnd.djvu (https://www.iana.org/assignments/media-types/image/vnd.djvu)
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