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

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

---

Migrated from T1315 (view on Phabricator)

added Web app priority:Low labels

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!

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

assigned to @anlambert

added priority:Normal label and removed priority:Low label

changed title 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

mentioned in commit 95facd24

closed

@zack, This is now deployed