Sirherobrine23/Jellyfin_Xtream
1
0
Fork 0
Code Issues 26 Pull Requests 6 Actions Releases Wiki Activity

Filter categories on the client to prevent rate limits #162

Merged
Kevinjil merged 1 commits from fix/livetv-concurrency into master 2025-02-26 20:18:05 +00:00
Conversation 5 Commits 1 Files Changed 4 +11 -10
Kevinjil commented 2025-02-24 21:33:24 +00:00 (Migrated from github.com)
Copy Link

Potential fix for #157

Potential fix for #157
EpicOfficer commented 2025-02-26 19:05:45 +00:00 (Migrated from github.com)
Copy Link

Pretty sure this will fix my issue, you got a manifest link I can use to test?

I get a 403 error server side loading channels etc, and switching to an alternate url for the same provider temporarily worked for one request, then gone... maybe ratelimit??

Pretty sure this will fix my issue, you got a manifest link I can use to test? I get a 403 error server side loading channels etc, and switching to an alternate url for the same provider temporarily worked for one request, then gone... maybe ratelimit??
Kevinjil commented 2025-02-26 19:52:43 +00:00 (Migrated from github.com)
Copy Link

@EpicOfficer see https://github.com/Kevinjil/Jellyfin.Xtream/issues/157#issuecomment-2679704109 and following comments. Feedback is very appreciated :)

@EpicOfficer see https://github.com/Kevinjil/Jellyfin.Xtream/issues/157#issuecomment-2679704109 and following comments. Feedback is very appreciated :)
EpicOfficer commented 2025-02-26 20:12:41 +00:00 (Migrated from github.com)
Copy Link

Beautiful!! Thank you! This worked straight away, good work

Beautiful!! Thank you! This worked straight away, good work
Kevinjil commented 2025-02-26 20:23:53 +00:00 (Migrated from github.com)
Copy Link

I for sure hate providers returning the wrong HTTP status codes...

I for sure hate providers returning the wrong HTTP status codes...
EpicOfficer commented 2025-02-26 20:27:46 +00:00 (Migrated from github.com)
Copy Link

As a developer myself... I wholeheartedly agree. Should've been a 429

On Wednesday, 26 February 2025 at 20:24, Kevin Jilissen @.***> wrote:

I for sure hate providers returning the wrong HTTP status codes...

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

[Kevinjil]Kevinjil left a comment (Kevinjil/Jellyfin.Xtream#162)

I for sure hate providers returning the wrong HTTP status codes...

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

As a developer myself... I wholeheartedly agree. Should've been a 429 On Wednesday, 26 February 2025 at 20:24, Kevin Jilissen ***@***.***> wrote: > I for sure hate providers returning the wrong HTTP status codes... > > — > Reply to this email directly, [view it on GitHub](https://github.com/Kevinjil/Jellyfin.Xtream/pull/162#issuecomment-2686111407), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/ACHR7QXTB4C5DLW4E4MNRED2RYPG7AVCNFSM6AAAAABXZBZXAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOBWGEYTCNBQG4). > You are receiving this because you were mentioned.Message ID: ***@***.***> > > [Kevinjil]Kevinjil left a comment [(Kevinjil/Jellyfin.Xtream#162)](https://github.com/Kevinjil/Jellyfin.Xtream/pull/162#issuecomment-2686111407) > > I for sure hate providers returning the wrong HTTP status codes... > > — > Reply to this email directly, [view it on GitHub](https://github.com/Kevinjil/Jellyfin.Xtream/pull/162#issuecomment-2686111407), or [unsubscribe](https://github.com/notifications/unsubscribe-auth/ACHR7QXTB4C5DLW4E4MNRED2RYPG7AVCNFSM6AAAAABXZBZXAWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMOBWGEYTCNBQG4). > You are receiving this because you were mentioned.Message ID: ***@***.***>
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
blocked
This PR is blocked due to a commented reason
 breaking
This PR introduces breaking changes
 bug
This PR or Issue describes or fixes something that isn't working
 chore
This PR contains repository maintenance changes
 ci
This PR updates or changes something CI related
 confirmed
This Issue describes a bug that has been confirmed by a org member
 dependencies
This PR updates or changes a dependency file
 deprecated
This PR marks a feature, function or other public API element as deprecated
 documentation
This PR or Issue aims to improve or add to documentation
 duplicate
This Issue or PR is a duplicate of an existing Issue or PR
 feature
This PR or Issue requests or introduces a new feature
 github-actions
This PR updates or changes GH Actions used in this repo
 good first issue
This Issue contains a requirement easy to be picked up by project newcomers
 help wanted
This Issue or PR requires extra attention and/or requires external help
 invalid
This Issue describes an invalid request or bug report
 major-bug
This PR fixes a major bug
 major-feature
This PR introduces a major new feature
 nuget
This PR updates or changes some NuGet dependencies
 question
This Issue contains a question
 release-prep
This PR is a meta PR to prepare for the next release
 removed
This PR removes a feature, function or other public API element
 reverted
This PR reverts a commit or multiple commits
 skip-changelog
This PR gets ignored by the Release Drafter workflow
 tests
This PR adds or imposes tests (e.g. coverage, quality, ...)
 wontfix
This Issue contains a request or 'bug' that wont be fixed (e.g. out-of-scope, not-an-issue, ...)
No Label
Milestone
No items
No Milestone
Assignees
Clear assignees
Sirherobrine23
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Sirherobrine23/Jellyfin_Xtream#162
No description provided.
Delete Branch "fix/livetv-concurrency"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?