Skip to content

fix: resolve BDSM Singles browser challenge - #520

Open
kristoisberg wants to merge 2 commits into
kaifcodec:mainfrom
kristoisberg:fix/bdsmsingles-username-module-errors
Open

fix: resolve BDSM Singles browser challenge#520
kristoisberg wants to merge 2 commits into
kaifcodec:mainfrom
kristoisberg:fix/bdsmsingles-username-module-errors

Conversation

@kristoisberg

Copy link
Copy Markdown
Collaborator

BDSM Singles has introduced a browser challenge, which caused the username module to fail. This PR solves the challenge to make the module work again.

PS: Could be another regional thing. If the module previously worked for any of you, and this PR broke something, please let me know.

@kaifcodec kaifcodec added the review required Review pending or needs a review again label Aug 8, 2026
@brunolm

brunolm commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

There's a conflict, but LGTM

@kaifcodec

Copy link
Copy Markdown
Owner

@brunolm I think there's no need to merge it or is there?
You already had opened a pr that has been merged #556 , Is here something in this that is needed to be merged?

@brunolm

brunolm commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

@kristoisberg can you verify if main works for you and close this if not needed?

# Conflicts:
#	user_scanner/user_scan/adult/bdsmsingles.py
@kristoisberg
kristoisberg requested a review from brunolm August 12, 2026 13:46
@kristoisberg

Copy link
Copy Markdown
Collaborator Author

I resolved the merge conflicts.

@brunolm I re-added the check for a 302 error code, interpreting it as an available username. The comment in your patch described two different scenarios where a 302 error code could occur (available username and browser challenge), this PR resolves the latter, only keeping available username as a (known) option. Thoughts?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review required Review pending or needs a review again

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants