Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Mrss icon format extensions #4416

Closed
Phantop opened this issue Jan 24, 2025 · 1 comment
Closed

Mrss icon format extensions #4416

Phantop opened this issue Jan 24, 2025 · 1 comment
Labels
Bug-Report Confirmed bug report

Comments

@Phantop
Copy link
Contributor

Phantop commented Jan 24, 2025

While working on a new bridge, I noticed that Mrss requires icon urls to end in an extension. I'm curious as to the justification for this and whether it is necessary, as testing a feed with an icon url with no extension has had it validate succesfully with the validators listed at the top of the file, as well as accepted with the icon showing up in Miniflux.

Is it possible this requirement could be removed?

@Phantop Phantop added the Bug-Report Confirmed bug report label Jan 24, 2025
dvikan added a commit to dvikan/rss-bridge that referenced this issue Jan 25, 2025
@dvikan
Copy link
Contributor

dvikan commented Jan 25, 2025

i agree it's too strict

@dvikan dvikan closed this as completed in 58b3cfb Jan 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug-Report Confirmed bug report
Projects
None yet
Development

No branches or pull requests

2 participants