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

[DO-NOT-MERGE] Recaptcha public review #2128

Closed
wants to merge 12 commits into from

Conversation

Xiaoshouzi-gh
Copy link
Contributor

This is a rebased branch against master with recaptcha features.
The changes has all been previously reviewed and approved.

This is the branch we will merge once we are ready for release.

Xiaoshouzi-gh and others added 11 commits March 2, 2023 14:27
* Defined reCAPTCHA config.
 - Added reCAPTCHA protection states.
 - Added reCAPTCHA action rule.
 - Added reCAPTCHA key config.
* Support reCaptcha config /create update on tenants.
 - Support create and update tenants with reCaptcha config.
 - Added reCaptcha unit tests on tenants operations.
* Recaptcha config changes in project config.
- Implemented getProjectConfig.
- Implemented updateProjectConfig.
- Updated error code.
- Add Term of Service consents.
* Added integ test for Project Config and Tenants update on reCAPTCHA config
* Support use_account_defender add-on feature for reCAPTCHA config.
* Added integration test for account defender feature
- Fix merge conflicts with sms region change
- Do not throw exceptions enabling account defender when recaptcha is OFF
@Xiaoshouzi-gh Xiaoshouzi-gh added the release:stage Stage a release candidate label Mar 29, 2023
@Xiaoshouzi-gh Xiaoshouzi-gh marked this pull request as ready for review March 29, 2023 20:36
@Xiaoshouzi-gh Xiaoshouzi-gh mentioned this pull request Mar 29, 2023
@Xiaoshouzi-gh Xiaoshouzi-gh changed the title Recaptcha public review [DO-NOT-MERGE] Recaptcha public review Mar 29, 2023
@Xiaoshouzi-gh
Copy link
Contributor Author

cc @prameshj @renkelvin

@Xiaoshouzi-gh
Copy link
Contributor Author

duplicate pr of #2129.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:stage Stage a release candidate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant