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

Update to Recaptcha Enterprise #1128

Merged
merged 11 commits into from
Jan 14, 2025
Merged

Update to Recaptcha Enterprise #1128

merged 11 commits into from
Jan 14, 2025

Conversation

wrandall22
Copy link
Contributor

This PR switches from Recaptcha v3 to Recaptcha Enterprise. The scoring has moved from an AEM API call to the Cortex Gateway, so we will not know it in real time, but on gift submission. We also changed the actions to match up to one pre-existing Google defined action and a similar wording for branded.

@wrandall22 wrandall22 force-pushed the recaptcha-enterprise branch from 517e012 to 0a13f23 Compare January 7, 2025 14:56
@wrandall22 wrandall22 requested a review from canac January 8, 2025 18:58
@wrandall22
Copy link
Contributor Author

Plan is to go live on Tuesday 1/14 at 2pm.

@wrandall22 wrandall22 force-pushed the recaptcha-enterprise branch from 057c4e4 to 4464058 Compare January 9, 2025 14:16
Copy link
Contributor

@canac canac left a comment

Choose a reason for hiding this comment

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

Looks great! I see the Recaptcha token and action in the request in staging for regular checkout and branded checkout.

@wrandall22 wrandall22 force-pushed the recaptcha-enterprise branch from c1bb967 to f27eb23 Compare January 9, 2025 15:57
@wrandall22 wrandall22 force-pushed the recaptcha-enterprise branch from f27eb23 to 2dc3a44 Compare January 14, 2025 18:37
@wrandall22 wrandall22 merged commit 603c396 into master Jan 14, 2025
4 checks passed
@wrandall22 wrandall22 deleted the recaptcha-enterprise branch January 14, 2025 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants