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

[IMPROVEMENT] Add ability to enter multiple captchas before posting, use entered captchas until they expire #252

Closed
K1rakishou opened this issue Jul 28, 2019 · 3 comments · Fixed by #253
Assignees
Labels
enhancement New feature or request

Comments

@K1rakishou
Copy link

K1rakishou commented Jul 28, 2019

UPD: For some reason one captcha cannot be used to make multiple posts right now. 4chanx just removes a captcha after the reply. I dunno whether it depends on the board or 4chan does not support it anymore (I tested it on /g/ and /qa/). So, for now, one captcha will be used to make one post. But you still will be able to enqueue multiple captchas. Once I figure out why it does not work (and if it is still possible) I will make captchas to be used until expired rather than one per a post.

@K1rakishou K1rakishou added the enhancement New feature or request label Jul 28, 2019
@K1rakishou K1rakishou self-assigned this Jul 28, 2019
@K1rakishou K1rakishou changed the title [IMPROVEMENT] Add ability to enter multiple captchas before posting, use entered captcha until they expire [IMPROVEMENT] Add ability to enter multiple captchas before posting, use entered captchas until they expire Jul 28, 2019
K1rakishou added a commit that referenced this issue Jul 28, 2019
@Adamantcheese
Copy link
Owner

Should I wait to pull this before you figure that issue out?

@K1rakishou
Copy link
Author

I think, yes, let's wait a little bit.

@K1rakishou
Copy link
Author

K1rakishou commented Jul 29, 2019

So, after a little bit of investigation, it looks like using one captcha for multiple posts is only possible with JS captcha. Doesn't work with forced (via 4chanx) no-js captcha. Nothing can be done about it.
Don't merge this yet, though, because there is a tiny bug left that needs to be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants