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

fix(recaptcha): add safeguards against duplicate recaptcha rendering #3624

Merged
merged 1 commit into from
Dec 13, 2024

Conversation

chickenn00dle
Copy link
Contributor

@chickenn00dle chickenn00dle commented Dec 12, 2024

All Submissions:

Changes proposed in this Pull Request:

Closes https://app.asana.com/0/1207817176293825/1208950357031079/f

This PR adds some safeguards to protect against potential duplicate rendering in recaptcha logic. We also make the recaptcha message a bit clearer to indicate the problem is connecting to recaptcha.

How to test the changes in this Pull Request:

  1. Smoke test the registration and checkout flows

Other information:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your changes, as applicable?
  • Have you successfully ran tests with your changes locally?

@chickenn00dle chickenn00dle added the [Status] Needs Review The issue or pull request needs to be reviewed label Dec 12, 2024
@chickenn00dle chickenn00dle requested a review from a team as a code owner December 12, 2024 19:44
@chickenn00dle chickenn00dle changed the title fix(recaptcha): add safeguards for race conditions fix(recaptcha): add safeguards against duplicate recaptcha rendering Dec 12, 2024
Copy link
Contributor

@dkoo dkoo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It doesn't stop the alerts from happening on my test site, but the improved message does help.

@github-actions github-actions bot added [Status] Approved The pull request has been reviewed and is ready to merge and removed [Status] Needs Review The issue or pull request needs to be reviewed labels Dec 13, 2024
@chickenn00dle chickenn00dle merged commit d95ee7e into release Dec 13, 2024
11 checks passed
@chickenn00dle chickenn00dle deleted the hotfix/add-recaptcha-safeguards branch December 13, 2024 19:48
matticbot pushed a commit that referenced this pull request Dec 13, 2024
## [5.9.2](v5.9.1...v5.9.2) (2024-12-13)

### Bug Fixes

* **recaptcha:** add safeguards against duplicate renders ([#3624](#3624)) ([d95ee7e](d95ee7e))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 5.9.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

matticbot pushed a commit that referenced this pull request Dec 16, 2024
# [5.10.0-alpha.2](v5.10.0-alpha.1...v5.10.0-alpha.2) (2024-12-16)

### Bug Fixes

* dont load textdomain too early ([#3629](#3629)) ([76c1f97](76c1f97))
* hide duplicate notices if all was dismissed ([#3630](#3630)) ([cf48188](cf48188))
* **recaptcha:** add safeguards against duplicate renders ([#3624](#3624)) ([d95ee7e](d95ee7e))
@matticbot
Copy link
Contributor

🎉 This PR is included in version 5.10.0-alpha.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ras-acc testing released on @alpha released [Status] Approved The pull request has been reviewed and is ready to merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants