Sounds good.

There are missing tests which will be added later in a future diff.

Quoting from seirl's swh-devel email:

I only did manual tests for now, i'm currently
adapting the unit tests I wrote for the old version. That'll come in a
future diff.

Note: as build fails, i've manually arc patch locally and tests are still good.

Looks good to me too.

Just one thing to discuss on, I am wondering if it is coherent to mount the vault feature under the api endpoint (current uri scheme is /api/1/vault/....).
Using simply /vault/.... and creating a dedicated Django application (like browse) could be another option.
@zack , what is your opinion on this ?