diff --git a/src/__tests__/field/captcha/third_party_captcha.test.jsx b/src/__tests__/field/captcha/third_party_captcha.test.jsx index fcff12d97..5cc1449b3 100644 --- a/src/__tests__/field/captcha/third_party_captcha.test.jsx +++ b/src/__tests__/field/captcha/third_party_captcha.test.jsx @@ -210,7 +210,7 @@ describe('ThirdPartyCaptcha', () => { }); }); - it('should retry 3 times on error and then set value to BYPASS_CAPTCHA dummy token for failOpen', () => { + it('should retry 3 times on error for failOpen', () => { const renderParams = global.window.turnstile.render.mock.calls[0][1]; for (let i = 0; i < 3; i++) { const renderParams = global.window.turnstile.render.mock.calls[i][1]; @@ -227,7 +227,6 @@ describe('ThirdPartyCaptcha', () => { const { onChange } = wrapper.props; expect(onChange.mock.calls).toHaveLength(1); - expect(onChange.mock.calls[0][0]).toBe('BYPASS_CAPTCHA'); }); }); diff --git a/src/field/captcha/third_party_captcha.jsx b/src/field/captcha/third_party_captcha.jsx index cfe3eee66..92c29aeff 100644 --- a/src/field/captcha/third_party_captcha.jsx +++ b/src/field/captcha/third_party_captcha.jsx @@ -123,7 +123,7 @@ export class ThirdPartyCaptcha extends React.Component { this.props.onChange(value); this.props.onErrored(); }); - }; + }; } getRenderParams() { @@ -157,9 +157,6 @@ export class ThirdPartyCaptcha extends React.Component { this.setState(prevState => ({ retryCount: prevState.retryCount + 1 })); - } else { - // similar implementation to ARKOSE_PROVIDER failOpen - this.changeHandler('BYPASS_CAPTCHA'); } return true; } @@ -167,7 +164,7 @@ export class ThirdPartyCaptcha extends React.Component { } return renderParams; } - + injectCaptchaScript(callback = noop) { const { provider, hl, clientSubdomain, sitekey } = this.props; const callbackName = `${providerDomPrefix(provider)}Callback_${Math.floor(Math.random() * 1000001)}`; @@ -189,7 +186,9 @@ export class ThirdPartyCaptcha extends React.Component { return; } removeScript(scriptUrl); - this.changeHandler('BYPASS_CAPTCHA'); + if (provider === ARKOSE_PROVIDER) { + this.changeHandler('BYPASS_CAPTCHA'); + } }; window[callbackName] = arkose => { callback(arkose); @@ -252,7 +251,7 @@ export class ThirdPartyCaptcha extends React.Component { reset() { const provider = getCaptchaProvider(this.props.provider); - if (this.props.provider === FRIENDLY_CAPTCHA_PROVIDER) { + if (this.props.provider === FRIENDLY_CAPTCHA_PROVIDER) { if (this.widgetInstance) { this.widgetInstance.reset(); }