huacnlee / rucaptcha

Captcha Gem for Rails, which generates captcha image by Rust.
https://huacnlee.github.io/rucaptcha
MIT License
695 stars 133 forks source link

feat(check-captcha-expired): add helper 'is_expired_rucaptcha?' to ch… #72

Closed ethan-well closed 5 years ago

ethan-well commented 5 years ago

添加 helper 方法 is_expired_rucaptcha? 用于判断验证码是否过期:

现在的 verify_rucaptcha? 方法,在用户输入错误或者验证码超时两种情况下返回的都是 false,但是有种场景:验证码已经过期,用户填写内容和前端显示的验证码看起来一致,不能明确提示用户是输入错误还是已经过期。所以我添加了 is_expired_rucaptcha? 方法,用于判断验证码是否过期。

ethan-well commented 5 years ago

@huacnlee 你好,请问这个 PR 有什么问题吗?一直没有被处理。

huacnlee commented 5 years ago

这个需求没意义,验证码过期设定时间都是合理的,过期也是验证码错误,在这种场景只需要简单提示验证码不正确即可,无需解释那么详细,那样做只会让事情复杂化。

ItsWewin notifications@github.com于2019年6月14日 周五上午12:15写道:

@huacnlee https://github.com/huacnlee 你好,请问这个 PR 有什么问题吗?一直没有被处理。

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/huacnlee/rucaptcha/pull/72?email_source=notifications&email_token=AAABLDRSOHC5EVLTG4IBQHDP2JXB7A5CNFSM4GYTRJAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXUG6XI#issuecomment-501772125, or mute the thread https://github.com/notifications/unsubscribe-auth/AAABLDTTOFC67CPCXTCZL3TP2JXB7ANCNFSM4GYTRJAA .

ethan-well commented 5 years ago

我觉得意义在于没过期的情况下提示用户重新输入,过期的情况提示用户刷新验证码,这样对于那种已经过期的验证码不至于让用户产生 "明明输入没有错,为什么提示验证码错误" 的疑问。