Skip to content

fix login CAPTCHA#439

Merged
briskt merged 5 commits into
mainfrom
captcha
Jun 29, 2026
Merged

fix login CAPTCHA#439
briskt merged 5 commits into
mainfrom
captcha

Conversation

@briskt

@briskt briskt commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

IDP-2168 ssp-base CAPTCHA broken


Fixed

  • Fixed the login CAPTCHA. Previous template changes added buttons to the page, breaking the non-specific 'button' selector.

@briskt
briskt requested a review from a team as a code owner June 26, 2026 22:26
@briskt
briskt requested review from ethancanne, forevermatt, jason-jackson and mtompset and removed request for a team June 26, 2026 22:26
Comment thread modules/silauth/src/Auth/Source/auth/Authenticator.php
@briskt
briskt merged commit 5668920 into main Jun 29, 2026
2 checks passed
@briskt
briskt deleted the captcha branch June 29, 2026 16:21
@ethancanne

Copy link
Copy Markdown
Contributor

@briskt does this fix the issue you mentioned here: #436 (comment)? If so, have you already carried out a production deployment?

@briskt

briskt commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

@briskt does this fix the issue you mentioned here: #436 (comment)? If so, have you already carried out a production deployment?

Yes, it does and it has been deployed to all but the usa IdP. I'll do that now.

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants