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

Fix: play auth + get_streams 504 #36

Closed
wants to merge 5 commits into from
Closed

Conversation

redref
Copy link

@redref redref commented Nov 2, 2023

TODO :

  • fix logos

@redref redref marked this pull request as draft November 2, 2023 13:28
@redref redref mentioned this pull request Nov 2, 2023
@remzouille
Copy link
Contributor

Il y a un problème dans ta fonction get_stream_info : tu fais systématiquement deux requêtes vers endpoint_stream_info alors que ça n'est nécessaire que lorsqu'il faut régénérer l'authentification.

@redref
Copy link
Author

redref commented Nov 2, 2023

Il y a un problème dans ta fonction get_stream_info : tu fais systématiquement deux requêtes vers endpoint_stream_info alors que ça n'est nécessaire que lorsqu'il faut régénérer l'authentification.

Thanks, this is fixed.

@redref
Copy link
Author

redref commented Nov 2, 2023

Current version working for now. Will test until tomorrow to see if I can get the license 401 I was getting earlier.

For channel logos, I can see them in devtools only on the main page which is HTML, would need BeautifulSoup to parse it, but we could then filter not subscribed channels.

@f-lawe f-lawe closed this Jun 13, 2024
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.

None yet

3 participants