-
Notifications
You must be signed in to change notification settings - Fork 8
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add timeout to Resource #15
Comments
what kind of timeout ? |
the read/response timeout: request.timeout( ). (I would not fail the load but report the timeout in Resource.Info) |
I'm not sure about this. We typically send a "resource" timeout, meaning the whole tree. Having a per-request/response timeout is fine, but how does it apply to the whole tree ? Right now there is no timeout, as far as I can see. Where did you get that 60s value ? |
Oh correct. I missed the point about Resource can be a whole tree. |
per default timeout is 60000. Maybe we could make it configurable at resource level.
@sbordet WDYT?
The text was updated successfully, but these errors were encountered: