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.

Sign up for GitHub

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

Jump to bottom

files: check download of some files #2064

Closed
tiborsimko opened this issue Dec 9, 2017 · 1 comment · Fixed by #2077
Closed

files: check download of some files #2064

tiborsimko opened this issue Dec 9, 2017 · 1 comment · Fixed by #2077
Assignees
@tiborsimko
Labels
Topic: files
Milestone
COD3-Production

Comments

@tiborsimko
Copy link
Member

As observed in #2055 (comment), some file download works, some does not.

@ioannistsanaktsidis This is without doubt also why the embedded previewer worked for some records and why it did not work for others, as I mentioned you IRL last week...
@tiborsimko tiborsimko added this to the COD3-Production milestone Dec 9, 2017
@ioannistsanaktsidis
Copy link
Contributor

Some files are missing the sha1:... property in the fixtures thats why download is failing.

tiborsimko added a commit to tiborsimko/opendata.cern.ch that referenced this issue Dec 11, 2017
@tiborsimko
records: sha1 checksum fix for files
ed77a4c 
* Fixes checksum information for several CMS files. (closes cernopendata#2064)

Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
@tiborsimko tiborsimko mentioned this issue Dec 11, 2017
Merged
tiborsimko added a commit to tiborsimko/opendata.cern.ch that referenced this issue Dec 11, 2017
@tiborsimko
records: sha1 checksum fix for files
78f3b65 
* Fixes checksum information for several CMS files. (closes cernopendata#2064)

Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tiborsimko tiborsimko

Labels
Topic: files
Projects
None yet
Milestone
COD3-Production
Development

Successfully merging a pull request may close this issue.

records: sha1 checksum fix for files tiborsimko/opendata.cern.ch
2 participants
@tiborsimko @ioannistsanaktsidis