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

Add support for Recaptcha enterprise #756

Merged

Conversation

Lakshan-Banneheke
Copy link
Contributor

@Lakshan-Banneheke Lakshan-Banneheke commented Sep 7, 2023

Proposed changes in this pull request

This is a continuation of this PR: #684

This PR is for adding the support for reCaptcha enterprise.

If reCaptcha enterprise is enabled, the verification method is different from v2 and v3. This is mainly handled in the CaptchaUtil.
Additional variables needed for reCaptcha enterprise support are added.

Related Issues

When should this PR be merged

  • Can be merged now

Follow up actions

  • Update the frontend for recaptcha enterprise

@CLAassistant
Copy link

CLAassistant commented Sep 7, 2023

CLA assistant check
All committers have signed the CLA.

hwupathum
hwupathum previously approved these changes Sep 8, 2023
Copy link
Contributor

@hwupathum hwupathum left a comment

Choose a reason for hiding this comment

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

Will need to refactor the code when other types of catpcha are introduced

@Lakshan-Banneheke Lakshan-Banneheke force-pushed the recaptcha-enterprise branch 2 times, most recently from 645e44c to d603ed2 Compare September 11, 2023 04:34
@Buddhimah Buddhimah merged commit b4b3081 into wso2-extensions:master Sep 15, 2023
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.

8 participants