Remove unused parameters on POST /api/upload
There are many query arguments specific to POST /api/upload
, and while some are important, e.g the admin
query arg to remove virus/limit checks, I don't see much case on having to support arguments such as domain
, subdomain
when we can fetch from settings (and settings fetch happens most of the time anyways), or the random
argument for a random domain (that is practically unused by people, but that's because its undocumented, and I don't think anyone would use it if it was documented anyways).
This complicates how we decide on which domain to select since we have 3 ways to do it. They are undocumented, but I'll still categorize this as an api breakage and so, scheduled for V3. Feel free to argue for them. Reducing complexity is the goal here.
Feedback very wanted, from coredevs and users alike.