Specify Captcha support
Depend on https://github.com/authgear/authgear-server/pull/4399
@chpapa @pkong-ds I updated the spec according to the comments we had in the product meeting. Please have a look when you are free. cc @tung2744 @fungc-io
@chpapa @pkong-ds I have updated the spec once again. The major changes are
- Rename
captchatobot_protection - Place
risk_assessmentunderbot_protection -
providersbecomesprovider, that is, we support only one provider at a time.
Configuration example
bot_protection:
enabled: true
provider:
type: cloudflare
site_key: "SITE_KEY"
risk_assessment:
enabled: true
provider:
type: recaptchav3
site_key: "SITE_KEY"
risk_score:
low: 0.2
medium: 0.5
high: 0.7
authentication_flow:
login_flows:
- name: default
steps:
- type: identify
one_of:
- identification: email
- type: authenticate
one_of:
- authentication: primary_password
bot_protection:
mode: "risk_level_high"
provider:
type: cloudflare
risk_assessment:
enabled: true
provider:
type: recaptchav3
@louischan-oursky seems I don't have access to push to louischan-oursky/dev-1352-specify-captcha. Submitted a PR to your branches instead LOL. ref https://github.com/louischan-oursky/authgear-server/pull/1
Please advise if otherwise~ 🙏
@pkong-ds I added your commit to this PR. And also added a commit to update the audit log event name.
@pkong-ds I added a commit based on your changes.
@chpapa @tung2744 @fungc-io I added a commit to specify the configuration to support the behavior we discussed in the product meeting. https://github.com/authgear/authgear-server/pull/4282/commits/b8c90dcd7c056153ef7e821d812a53789d36d2aa#diff-7fa5fe90ad92ac79b786c50d705b525113599d205b1268f4d093a4249e8aeb03R60
@louischan-oursky added commit for new agreed spec 🙏 https://github.com/authgear/authgear-server/pull/4282/commits/dec4edc15245a63f3f1cc11013959132cf97a96f
Preview
@louischan-oursky added 2 commits to for product meeting discussion result
@tung2744 This should be good to merge.