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

[Local] Skip media type if it's invalid #750

Merged
merged 1 commit into from
Jan 19, 2025
Merged

[Local] Skip media type if it's invalid #750

merged 1 commit into from
Jan 19, 2025

Conversation

jocmp
Copy link
Owner

@jocmp jocmp commented Jan 19, 2025

Fixes the following error when adding a feed that sends back the wrong content type.

2025-01-19 16:46:24.146 12335-29945 cr.local_account.find com.capyreader.app.debug I error_message=Parameter is not formatted correctly: "application/rss+xml; charset=utf-8" for: "text/xml; application/rss+xml; charset=utf-8"

The fix is to call toMediaTypeOrNull instead of toMediaType which will throw.

@jocmp jocmp merged commit 2bf7c5f into main Jan 19, 2025
1 check passed
@jocmp jocmp deleted the jc/748/media-type branch January 19, 2025 21:57
jocmp added a commit that referenced this pull request Jan 19, 2025
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.

1 participant