-
Notifications
You must be signed in to change notification settings - Fork 62
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
Document backends (HTTP and Downloads) #426
Comments
Somethings it might be good to document
|
Great list!
If it's in the
I think there is not a safe way to change the global backend from within library code, and libraries should set the backend per-request if they need a feature from a particular one, and otherwise should respect the default. I think "applications", i.e. leafs of the package dependency tree (so they themselves have no downstream packages, only upstream ones they depend on) can set the backend to whatever they would like. Ideally, if one backend is strictly better (i.e. there's no reason to need another one), we should change the default here in AWS.jl. I think the DownloadsBackend is there theoretically (I don't think the HTTP.jl backend does anything the DownloadsBackend can't) but could possibly use some further testing in the wild first-- I've been using it and recommending it at Beacon without issues so far though, but mostly just with s3 and auth. But if we somehow end up at a point where there are two backends and you might want one for one thing and another for another thing, then we can't really solve the issues by changing the default. I think the solution overall then is to just do it per-request when you need a particular one. |
For high-level API calls we can use #457 to allow for different libraries to specify what backend to use. |
I’ve just been trying and failing to work out how to switch to the Downloads backend. I was told once but can’t find where. So addressing this issue would be appreciated! Even just posting how to here would help. |
Found a way in #516 (comment)
|
These probably should get a page in the docs. Follow up to #396 and #401
The text was updated successfully, but these errors were encountered: