initialize data url alias for data.astropy.org#9187
initialize data url alias for data.astropy.org#9187dstndstn wants to merge 1 commit intoastropy:masterfrom
Conversation
pllim
left a comment
There was a problem hiding this comment.
Thanks, but unfortunately, this needs a bit more thought. First of all, the STScI server is no longer a thing (this just happened and was not widely advertised), so the mirror situation needs some figuring out among core maintainers. Secondly, this completely bypasses the config below, which concerns me.
Perhaps a less controversial approach is an option to disable alias caching. But I have not had time to think about this too much. And it looks like @aarchiba is onto something in her PRs.
|
If I understand the code's current logic, the point of the alias mechanism is to treat URLs that are requested from More to the point might be an ability for download_file to accept a list of mirrors, perhaps configured in the conffile. Then when Perhaps I'll try adding that to PR #9182 . It's not really the same topic but I can't easily make it a separate pull request. |
|
Actually, because this is a much easier decision to merge, why not just strip out the alias mechanism entirely? |
|
Superseded by #9182. Thanks anyway! |
To address #8676