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

Cached headers feature added #308

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

dmytrokoren
Copy link
Contributor

No description provided.

@dmytrokoren dmytrokoren changed the title Added cached headers Cached headers feature added Oct 2, 2024
@jdholtz
Copy link
Owner

jdholtz commented Nov 28, 2024

Hey @dmytrokoren, are these changes to help avoid 403/429s or just to improve efficiency of the program?

@dmytrokoren
Copy link
Contributor Author

Hey @dmytrokoren, are these changes to help avoid 403/429s or just to improve efficiency of the program?

This allows the use of successfully cached headers, which I believe can help reduce login issues and, in some cases, instances of 403 errors. However, it’s a 50/50 chance, as Cloudflare or similar detection software may unauthenticate the headers if they suspect bot activity.

@jdholtz
Copy link
Owner

jdholtz commented Nov 30, 2024

However, it’s a 50/50 chance, as Cloudflare or similar detection software may unauthenticate the headers if they suspect bot activity.

That’s what I’d be worried about when implementing a feature like this (or a similar situation where they detect the same headers being used for different accounts). Do you still think it’d overall be an improvement to the detection issues this script faces? It definitely would speed the script up, but perhaps not help with detection evasion much.

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