[PR #3835] [MERGED] Throw HttpException when tvdb sends us crap data #9719

Closed
opened 2026-02-07 06:07:17 +03:00 by OVERLORD · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/jellyfin/jellyfin/pull/3835
Author: @cvium
Created: 8/7/2020
Status: Merged
Merged: 8/10/2020
Merged by: @Bond-009

Base: masterHead: fix_tvdb_404_shit_nginx_failure


📝 Commits (1)

  • 23dfadd Throw HttpException when tvdb sends us crap data

📊 Changes

1 file changed (+12 additions, -0 deletions)

View changed files

📝 MediaBrowser.Providers/Manager/ProviderManager.cs (+12 -0)

📄 Description

Changes
tvdb will send html instead of a proper 404. This special snowflake case is now being handled like the rest of the 404 by throwing multiple exceptions.


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/jellyfin/jellyfin/pull/3835 **Author:** [@cvium](https://github.com/cvium) **Created:** 8/7/2020 **Status:** ✅ Merged **Merged:** 8/10/2020 **Merged by:** [@Bond-009](https://github.com/Bond-009) **Base:** `master` ← **Head:** `fix_tvdb_404_shit_nginx_failure` --- ### 📝 Commits (1) - [`23dfadd`](https://github.com/jellyfin/jellyfin/commit/23dfadd4300e251b737d4126d5e361921263113c) Throw HttpException when tvdb sends us crap data ### 📊 Changes **1 file changed** (+12 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `MediaBrowser.Providers/Manager/ProviderManager.cs` (+12 -0) </details> ### 📄 Description <!-- Ensure your title is short, descriptive, and in the imperative mood (Fix X, Change Y, instead of Fixed X, Changed Y). For a good inspiration of what to write in commit messages and PRs please review https://chris.beams.io/posts/git-commit/ and our documentation. --> **Changes** tvdb will send html instead of a proper 404. This special snowflake case is now being handled like the rest of the 404 by throwing multiple exceptions. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
OVERLORD added the pull-request label 2026-02-07 06:07:17 +03:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/jellyfin#9719