-
Notifications
You must be signed in to change notification settings - Fork 436
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
[YouTube] Potokens support implementation #1247
Conversation
87f3b37
to
f79abbf
Compare
caabbe9
to
fbee94e
Compare
Will Potoken support work for loading formats? |
fbee94e
to
5528d5c
Compare
5528d5c
to
163af60
Compare
The PoTokenProvider class is now expected to handle calls from multiple threads for thread safety.
163af60
to
b009e5b
Compare
b009e5b
to
d2cbd09
Compare
extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/PoTokenProvider.java
Show resolved
Hide resolved
extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/ClientsConstants.java
Show resolved
Hide resolved
extractor/src/main/java/org/schabi/newpipe/extractor/services/youtube/ClientsConstants.java
Show resolved
Hide resolved
@AudricV did you accidentally mark the comments of @FineFindus and me as resolved although they were not addressed or forgot to push something? |
Sorry for the late reply. No, it was fixed locally when I resolved the conversations. I thought I would be faster to push my changes, but this wasn't the case. This PR is superseded by #1272. |
Thanks to @AudricV for doing most of the work here! :)