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

Support geetest #47

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Support geetest #47

wants to merge 3 commits into from

Conversation

samc621
Copy link
Owner

@samc621 samc621 commented Aug 8, 2021

@bryank91 @pavog @labboy0276 @13ROY feel free to tag others who have been contributing to this repo...

I noticed a couple of weeks ago that the footsites starting using geetest instead of the recaptchaV2. 2Captcha has support for solving this captcha, so I added it, but am hitting some trouble.

After the catpcha is solved, I see the following page:
Screen Shot 2021-08-08 at 12 33 08 PM

If anyone has any ideas to get this across the line, that would be greatly appreciated.

This, along with the Nike issue (#28) are the most pressing bugs atm.

@Kohlsen
Copy link

Kohlsen commented Aug 16, 2021

Hey @samc621 I have been busy myself and haven't gotten to take a look at this just yet. Have you been able to test since you added support for geetest?

@samc621
Copy link
Owner Author

samc621 commented Aug 16, 2021

@Kohlsen every time I have tested, I got the same results (see screenshot above).

@xingfudeshi
Copy link

When this happens, you may need to add some retry mechanisms. @samc621

@xingfudeshi
Copy link

This occurs when the access speed is relatively fast. Wait a few seconds and try again. The problem is gone.

@samc621
Copy link
Owner Author

samc621 commented Oct 8, 2021

@xingfudeshi do you mean to wait before submitting the results? I can try that. Have you tested this?

@xingfudeshi
Copy link

I mean try resending the request or refreshing the page when you catch a 403 error.

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.

3 participants