Skip to content
This repository has been archived by the owner on Aug 17, 2021. It is now read-only.

Remove site key length check #221

Merged
merged 2 commits into from
Feb 26, 2018
Merged

Remove site key length check #221

merged 2 commits into from
Feb 26, 2018

Conversation

UltCombo
Copy link
Contributor

Recaptcha may generate site keys with more than 40 characters now, so the length check is outdated and can throw unexpected errors in such cases.

Recaptcha may generate site keys with more than 40 characters now, so the length check is outdated and can throw unexpected errors in such cases.
@UltCombo
Copy link
Contributor Author

I've had a Recaptcha site key with 41 characters and angular-recaptcha was throwing an error due to this length check. The length check seems superfluous so I believe it can be safely removed.

@coveralls
Copy link

coveralls commented Feb 26, 2018

Coverage Status

Coverage remained the same at 94.697% when pulling e240563 on UltCombo:patch-1 into a187773 on VividCortex:master.

@mtrias
Copy link
Contributor

mtrias commented Feb 26, 2018

Thanks @UltCombo @TheSharpieOne !

@iambrosi do you have some time to take care of releasing this? I don't have my NPM credentials at hand.

@iambrosi iambrosi merged commit 842d3c3 into VividCortex:master Feb 26, 2018
@iambrosi
Copy link
Contributor

This has been published as v4.1.3

@UltCombo UltCombo deleted the patch-1 branch February 27, 2018 04:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants