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

Add cn tests #87

Open
thesadru opened this issue Nov 13, 2022 · 4 comments
Open

Add cn tests #87

thesadru opened this issue Nov 13, 2022 · 4 comments
Labels
enhancement New feature or request help wanted Help from the community is needed to solve the issue

Comments

@thesadru
Copy link
Owner

I have cn cookies but I still probably cannot fix issues with cn unless someone tells me how. Dunno how useful it would be.

@thesadru thesadru added the enhancement New feature or request label Nov 13, 2022
@luoshuijs
Copy link

It is now very difficult to test on cn because the cn api now has captcha. If the server determines any abnormal behavior on an account, most apis will be unusable and return 1034 status code (which requires passing geetest). and even if it passes the captcha, the normal status only last about half a day.

@luoshuijs
Copy link

We currently don't know how "abnormal behavior" is defined. But we may be able to reduce the chance of running into the issue by:

  1. using the latest DS_SALT and User-Agent, and
  2. reduce the frequency of sending request to their API

@BachelorForever
Copy link
Contributor

I don't think it is practical to actually defeat geetest automatically.

Btw I have found an interesting header(X-Rpc-Device_id) in mobile hoyolab application.

I believe this header is somehow related to abnormal traffic detection, which we don't send this header at all.

@thesadru
Copy link
Owner Author

It might be a decent idea to add a bunch of non-essential query parameters and headers provided they are explicitly shown as non-essential in the code.

@thesadru thesadru added the help wanted Help from the community is needed to solve the issue label Dec 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request help wanted Help from the community is needed to solve the issue
Projects
None yet
Development

No branches or pull requests

3 participants