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

On channel watch error, set channel offline and try another #593

Closed
wants to merge 1 commit into from

Conversation

imranh2
Copy link

@imranh2 imranh2 commented Oct 28, 2024

Basically the title.

I had an issue where 1 channel "channel":"multiversus","channel_id":735931830 was causing me issues due to "error":"Content+Restricted+In+Region","error_code":"content_geoblocked","type":"error" and the whole app would just stop...

Bit of a hack as this will keep trying channels until it finds no more, but is that such a bad thing?

@imranh2 imranh2 changed the title On channel watch error, remove the channel and try another On channel watch error, set channel offline and try another Oct 28, 2024
@DevilXD
Copy link
Owner

DevilXD commented Oct 28, 2024

The problem is, set_offline won't work, because the viewer's update will set it back online like 1-2 minutes later. There's currently no solution nor way to "mark a channel as unwatchable, even if it's online". You'll need to exclude the game that causes this channel to appear, or edit the code to exclude this particular channel out.

I've expected this will appear sooner or later after I've pushed the last commit, but for now, cases like this are only reported as errors, but not handled.

@DevilXD DevilXD closed this Oct 28, 2024
@DevilXD DevilXD added the Fix This fixes an existing issue or error label Oct 28, 2024
@imranh2
Copy link
Author

imranh2 commented Oct 28, 2024

It's not just reported, the whole app stops due to exception handling.

It works pretty well for me, a channel fails due to whatever reason, gets marked as offline, and a new channel is picked. The state of the channel changing back to online isn't actually a issue becuase the miner doesn't switch to it.

Hard failing on a resolvable error in a program that you're supposed to let idle/run in the background? There's better ways than this...

@DevilXD
Copy link
Owner

DevilXD commented Oct 28, 2024

It's not just reported, the whole app stops due to exception handling.

Read my response under the actual issue here: #586 (comment)
The commit has been pushed: 3b4923a

The state of the channel changing back to online isn't actually a issue becuase the miner doesn't switch to it.

It will switch to it, if the channel is found to be of higher priority. That will happen if the channel being set online is the only channel for a higher priority game. This PR is not a fix for the issue in any remote way, and I won't accept it.

Hard failing on a resolvable error in a program that you're supposed to let idle/run in the background? There's better ways than this...

You are welcome to use a different software, or familiarize yourself with the project goals, where it's clearly stated that the errors are expected and will happen: https://github.com/DevilXD/TwitchDropsMiner#project-goals
The logic will need to be adjusted so that a channel can be marked as not possible to be watched when that happens.

@imranh2
Copy link
Author

imranh2 commented Oct 28, 2024

I missed your comment! I apologise.

The idea of unwatchable streams makes the most sense.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix This fixes an existing issue or error
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants