Skip to content
This repository has been archived by the owner on Jul 27, 2022. It is now read-only.

Make restbuilder timeouts configurable #23

Closed
chriserickson opened this issue Apr 27, 2018 · 0 comments · Fixed by #28
Closed

Make restbuilder timeouts configurable #23

chriserickson opened this issue Apr 27, 2018 · 0 comments · Fixed by #28
Assignees
Milestone

Comments

@chriserickson
Copy link

We have started running into an intermittent problem when verifying a captcha.
"Failed to connect to https://www.google.com/recaptcha/api/siteverify."

The error is bubbling up from com.megatome.grails.recaptcha.net.Post, which appears to have a hard-coded 1s read timeout. We suspect that there may be something in either our network or Google's servers that is causing the timeout to be exceeded.

It would help a lot of these timeouts were configurable to allow us to increase the timeout and see if it fixes the problem.

@iamthechad iamthechad self-assigned this Apr 28, 2018
@iamthechad iamthechad added this to the 3.2.0 milestone Apr 29, 2018
iamthechad added a commit that referenced this issue May 3, 2018
@ghost ghost added in progress and removed ready labels May 3, 2018
@ghost ghost added needs-review and removed in progress labels May 3, 2018
@ghost ghost removed the needs-review label May 5, 2018
iamthechad added a commit that referenced this issue May 6, 2018
Timeouts need to be set in the ctor, not as properties

Actually fixes #23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants