Page MenuHomeSoftware Heritage

Web API: lift the restriction that /raw only allows to download text-ish contents
Closed, MigratedEdits Locked

Description

It's probably time to review the policy in title.

Event Timeline

zack triaged this task as Low priority.Nov 12 2018, 11:09 AM
zack created this task.

I also think that we should remove that restriction, all contents should be retrievable through that endpoint.

Based on a discussion with @ardumont, this restriction was implemented before the requests rate limiting to the API.
Now that this rate limit is in place, I do not see why we should continue to have this restriction.

Moreover, it exists cases where the file command we are using to detect the mime type returns invalid results
(textual content detected as binary for instance).

@zack, what do you think ?

@anlambert: let's do this. Green light, go ahead!

(T1314 can hence probably be closed as invalid, but doing so will probably involve dropping some soon-to-be-dead code.)

anlambert renamed this task from Web API: lift the restriction that /raw only allows to download text-ish contents (?) to Web API: lift the restriction that /raw only allows to download text-ish contents.Nov 12 2018, 5:06 PM
anlambert claimed this task.
anlambert raised the priority of this task from Low to Normal.