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

[YouTube] Improve response status checks #271

Merged
merged 2 commits into from
Mar 1, 2020

Conversation

mauriciocolli
Copy link
Contributor

YouTube

  • Fail-fast if status exist and is anything other than "OK"

PeerTube

  • Update video used in age limit test (the previous was 404)

Incredible, forgot how a green check mark for tests looked like (though some areas are lacking tests).

@Stypox
Copy link
Member

Stypox commented Mar 1, 2020

Would it be possible that YouTube changes the "ok" string to something else (e.g. "success"), so that the extractor fails even though the content is there?

@wb9688
Copy link
Contributor

wb9688 commented Mar 1, 2020

"some areas"? A lot of areas ;) (reminds me that I still need to update #266…)

And yes, that green check mark is really nice.

@mauriciocolli
Copy link
Contributor Author

Would it be possible that YouTube changes the "ok" string to something else (e.g. "success"), so that the extractor fails even though the content is there?

Yes, but I didn't encounter this response and I tested a lot of videos.

It's easier to make an exception for "OK", than include all the others. But maybe it's worth it?


@TobiGr @wb9688 @Stypox

What about we upload some test videos like youtube-dl does, testing various parts? I think we could do some pretty reliable tests that way.

For example, tests for features, like premieres, could be done (checked and it can be set all the way to 2022, we could keep testing until then).

@wb9688
Copy link
Contributor

wb9688 commented Mar 1, 2020

@mauriciocolli: I think that's a good idea. However, we couldn't test everything that way, as some features aren't available to small channels and it's hard to test e.g. date parsing of a premiere that happened less than 24 hours ago.

@Stypox
Copy link
Member

Stypox commented Mar 1, 2020

Yes, but I didn't encounter this response and I tested a lot of videos.

Ok, then it's ok, and if youtube ever changes the "ok" string it would not take much to find the problem.

What about we upload some test videos like youtube-dl does, testing various parts?

Yeah, I thought about doing that, too. It is a good idea imo. I could do that if needed, I have some "experience" with YouTube (i.e. I used to run a stupid minecraft channel some years back). But should we publish the login data to a place everybody in TeamNewPipe can access, or what?

@Stypox
Copy link
Member

Stypox commented Mar 1, 2020

@wb9688 I have a channel with more that 4000 subscribers if needed (it is completely useless now, so if needed we could take advantage of it)

Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks

@TobiGr
Copy link
Contributor

TobiGr commented Mar 1, 2020

What about we upload some test videos like youtube-dl does, testing various parts? I think we could do some pretty reliable tests that way.

Good idea, but please keep in mind that you agree to YouTube's TOS when having a channel.

@TobiGr TobiGr merged commit 6446abc into TeamNewPipe:dev Mar 1, 2020
@mauriciocolli mauriciocolli deleted the improve-status-checking branch March 7, 2020 20:01
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.

4 participants