-
Notifications
You must be signed in to change notification settings - Fork 28
Add Image Captcha Solver #1612
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 Image Captcha Solver #1612
Conversation
✅ Deploy Preview for content-scope-scripts ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
Temporary Branch UpdateThe temporary branch has been updated with the latest changes. Below are the details:
Please use the above install command to update to the latest version. |
[Beta] Generated file diffTime updated: Thu, 10 Apr 2025 11:33:03 GMT Android
File has changed Integration
File has changed Windows
File has changed Apple
File has changed |
injected/src/features/broker-protection/captcha-services/providers/image.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/utils/token.js
Outdated
Show resolved
Hide resolved
Not sure why the auto respond PR steps are failing… |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! No blockers, just some recommendations
injected/src/features/broker-protection/captcha-services/providers/image.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/providers/image.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/utils/image.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/utils/token.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/utils/token.js
Outdated
Show resolved
Hide resolved
@brianhall I pushed a commit to fix it, hope that's ok. |
For sure. My editor must have stripped the extensions out, thank you! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM with one nit
injected/src/features/broker-protection/captcha-services/utils/token.js
Outdated
Show resolved
Hide resolved
injected/src/features/broker-protection/captcha-services/utils/token.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just for code ownership r+ maybe it's re-required after the file changes.
Asana Task/Github Issue:
Description
Asana: https://app.asana.com/0/72649045549333/1209395858706166/f
Testing Steps
Checklist
Please tick all that apply: