Skip to content
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

Timeout changes with glide 4.0.0 #2220

Closed
Sn0wfreezeDev opened this issue Aug 8, 2017 · 2 comments
Closed

Timeout changes with glide 4.0.0 #2220

Sn0wfreezeDev opened this issue Aug 8, 2017 · 2 comments
Milestone

Comments

@Sn0wfreezeDev
Copy link

I searched a long time for the solution how to easily change the Timeout from only 2.5s to something higher to 10s or 20s.
I did not find any real documentation about it, but I think it may be correct to use the Option class and it's memory function.

Using this code before the first Glide call is made will change the default timeout

Option.memory("com.bumptech.glide.load.model.stream.HttpGlideUrlLoader.Timeout", 2500).update(20_000, MessageDigest.getInstance("SHA-512"))

Glide Version: 4.0.0

Integration libraries: default

Device/Android Version: Android 6.0, Samsung Galaxy S5 and Android 7.0 on Emulator

Issue details / Repro steps / Use case background:

Glide load line / GlideModule (if any) / list Adapter code (if any):

Glide.with(view)
@sjudd
Copy link
Collaborator

sjudd commented Aug 8, 2017

Close, this isn't well documented.

I've added a section on the Option class to the documentation here: http://bumptech.github.io/glide/doc/options.html#component-options.

I'm also going to submit a change that will expose timeout() as a method on RequestOptions though with a note that it will only work if you're using Glide's default networking library.

I need to find a way to expose some of these individual options better in our documentation site, but I can't think of a great way at the moment other than just adding a hodgepodge of frequently mentioned options.

sjudd added a commit to sjudd/glide that referenced this issue Aug 8, 2017
@sjudd sjudd added this to the 4.1 milestone Aug 8, 2017
@sjudd
Copy link
Collaborator

sjudd commented Sep 1, 2017

If this comes up again I'll reconsider, hopefully the option added to RequestOptions is sufficient.

@sjudd sjudd closed this as completed Sep 1, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants