-
Notifications
You must be signed in to change notification settings - Fork 220
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 warn logs for recaptcha v3 and recaptcha enterprise #768
Add warn logs for recaptcha v3 and recaptcha enterprise #768
Conversation
f90a7dd
to
59e666b
Compare
...carbon.identity.captcha/src/main/java/org/wso2/carbon/identity/captcha/util/CaptchaUtil.java
Outdated
Show resolved
Hide resolved
.../org.wso2.carbon.identity.captcha.server.feature/resources/conf/captcha-config.properties.j2
Show resolved
Hide resolved
...carbon.identity.captcha/src/main/java/org/wso2/carbon/identity/captcha/util/CaptchaUtil.java
Outdated
Show resolved
Hide resolved
...carbon.identity.captcha/src/main/java/org/wso2/carbon/identity/captcha/util/CaptchaUtil.java
Outdated
Show resolved
Hide resolved
59e666b
to
8632c56
Compare
...carbon.identity.captcha/src/main/java/org/wso2/carbon/identity/captcha/util/CaptchaUtil.java
Outdated
Show resolved
Hide resolved
7c693bd
to
f9c7f75
Compare
PR builder started |
PR builder completed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Approving the pull request based on the successful pr build https://github.com/wso2/product-is/actions/runs/6452534435
Proposed changes in this pull request
This PR adds a threshold for warnings to be set in the deployment.toml and uses this thershold to issue warn logs if the score falls below the specified threshold. This is only applicable when recaptcha v3 or enterprise is enabled
When should this PR be merged
No pre-conditions
Related Issues
Follow up actions
N/A
Checklist (for reviewing)
General
Functionality
Code
Tests
Security
Documentation