Fail upload request early when possible

[Totktonada]

As I see from PR tarantool/mkrepo#27, mkrepo may fail metainformation update due to a malformed package. So, we can call those parse_deb() and parse_info() functions directly (before uploading files and calling mkrepo to generate the metainfo) and decline upload if something goes wrong: respond with a 4xx code.

This way a developer will receive a feedback about a malformed package (or a problem in rws) earlier.

[LeonidVas]

Yep.
I think this issue should be done after #2. To transactionally accept or reject a package.

[Totktonada]

AFAIU, now we report success for an upload request with any files (even if we mistakely request to upload something that is not a package). I found it very counter-intuitive.

[Totktonada]

I don't see a logical dependency on #2. We can validate all files first and then upload them non-transactionally. Those tasks look orthogonal.

[LeonidVas]

AFAIU, now we report success for an upload request with any files (even if we mistakely request to upload something that is not a package). I found it very counter-intuitive.

No. We check file extensions.

[LeonidVas]

I don't see a logical dependency on #2. We can validate all files first and then upload them non-transactionally. Those tasks look orthogonal.

IMHO we must to check all files and either upload all or not upload any file of the package.