Skip to content

reCaptcha#387

Merged
theScrabi merged 6 commits intoTeamNewPipe:masterfrom
be-neth:reCaptcha
Dec 24, 2016
Merged

reCaptcha#387
theScrabi merged 6 commits intoTeamNewPipe:masterfrom
be-neth:reCaptcha

Conversation

@be-neth
Copy link
Copy Markdown
Contributor

@be-neth be-neth commented Dec 7, 2016

The following patches try to implement reCaptcha from Youtube in a simple manner :

If HTTP 429 is returned by Downloader : reCaptchaException is thrown and reCatpchaAcitivity is started.
The Activity is a simple WebView (with Javascript enabled) displaying reCaptcha.

Ref #386 .

 * ReCaptchas are implemented using a simple WebView with Javascript enable.

 * All HTTP responses inside the WebView are catched (using onPageFinished())
 ** When Google reCatcha cookies are detected, register cookies to the
    Downloader class and Return to MainActivity
 * Fix typo setSearchWorkerResultListner to setSearchWorkerResultListener
@theScrabi
Copy link
Copy Markdown
Member

I was not able to test it correctly since I never had the case where I had to enter a captcha. Can you enter some screenshots, to see how it would look like :)

@theScrabi theScrabi merged commit 5cc1bbc into TeamNewPipe:master Dec 24, 2016
@be-neth
Copy link
Copy Markdown
Contributor Author

be-neth commented Dec 25, 2016

Hi,

Sure :

  1. After entering something in the search Bar
    screenshot_20161225-095036

  2. Captcha display in webview
    screenshot_20161225-095044

  3. Resolving Captcha
    screenshot_20161225-095110

  4. After Submit
    screenshot_20161225-095201

But there is still two issues with this pull request that i need to fix :

  1. After exiting the reCaptcha activity, I need to re-click on the search field to refresh the query.
  2. If I click from outside Newpipe Application on a Youtube link the reCaptcha exception is thrown but the activity is not started.

Do you plan to work on NewPipe at the 33c3 ? I will be there too :)

@theScrabi
Copy link
Copy Markdown
Member

Well why not :) Meet me at the NewPipe talk, so we can talk about that :)

This was referenced Sep 27, 2020
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.

2 participants