8000 [DO-NOT-MERGE] Recaptcha public review by Xiaoshouzi-gh · Pull Request #2128 · firebase/firebase-admin-node · GitHub
[go: up one dir, main page]

Skip to content

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

0