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

404 when downloading a GitHub release #179

Closed
nbigaouette opened this issue Oct 5, 2020 · 5 comments · Fixed by #180
Closed

404 when downloading a GitHub release #179

nbigaouette opened this issue Oct 5, 2020 · 5 comments · Fixed by #180

Comments

@nbigaouette
Copy link

In a project of mine I wrap a pre-built C library in a -sys crate (https://github.com/nbigaouette/onnxruntime-rs).

For better ergonomics I download the pre-built library in my build.rs using ureq (still on 1.4.0). This allows running tests against the library+wrapper in my CI (GitHub Actions). See: https://github.com/nbigaouette/onnxruntime-rs/blob/issue22/onnxruntime-sys/build.rs#L80-L87

The archive I want to download is onnxruntime-linux-x64-1.4.0.tgz from this release:
https://github.com/microsoft/onnxruntime/releases/tag/v1.4.0

The download used to work, but now it doesn't anymore. I specifically added this download so my GitHub Actions could have access to the library. Here's an example PR where the download passed: nbigaouette/onnxruntime-rs#21

Here's an example of failure: https://github.com/nbigaouette/onnxruntime-rs/runs/1191121056?check_suite_focus=true (part of PR nbigaouette/onnxruntime-rs#23 )

My build script prints the URL (https://github.com/nbigaouette/onnxruntime-rs/blob/issue22/onnxruntime-sys/build.rs#L210-L214) so I copy-pasted and used curl -LO https://... and it worked. GitHub releases uses a redirect to point to the real AWS S3 url to download. Maybe something changed in GitHub that broke how ureq is handling the redirects?

@jsha
Copy link
Collaborator

jsha commented Oct 6, 2020

Thanks for the report. I'm looking into it.

@jsha
Copy link
Collaborator

jsha commented Oct 6, 2020

Looks like we broke cross-host redirects as of 1.5.0. So sorry about that! Fix is in #180.

@algesten
Copy link
Owner

algesten commented Oct 6, 2020

@jsha i can make a quick 1.5.1. You decide whether #175 goes with it since it changes quite a bit in the pool.

@jsha jsha closed this as completed in #180 Oct 6, 2020
@jsha
Copy link
Collaborator

jsha commented Oct 6, 2020

Let's do the 1.5.1 without #175. Thanks!

@algesten
Copy link
Owner

algesten commented Oct 6, 2020

@nbigaouette 1.5.1 is on crates.io. Thanks for reporting!

nbigaouette added a commit to nbigaouette/onnxruntime-rs that referenced this issue Oct 6, 2020
<=1.5.0 fails to download pre-built archive.
See: algesten/ureq#179
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants