please explain a bit what this refactor is about in the commit message.

Also, this will "break the API". I am not opposed to this change, but some care should be taken when changing public's API behavior like this. (attempt to check if there are users of the behavior to be modified, document the new behavior, display/return a warning or something, etc)

In D7980#207563, @anlambert wrote:

Ack, I will make the change to the /api/1/origin/ endpoint to remove the URL processing then.

In D7980#207469, @anlambert wrote:

In D7980#207468, @jayeshv wrote:

In D7980#207464, @vlorentz wrote:

Why should this be implemented by the API (as opposed to UIs)?

This is part of our current REST APIs.

rebase

set timeout to 5s on both configs, as requested by anlambert

I've added the DescribeConfigs and AlterConfigs permissions for the mirror-test-rw principal, now I can do all I need.

typos in the ci msg

maybe only show a warning if the grant query fails (rather than crashing)?

lgtm (probably needs tedious hand-managed-db-migration-in-docker test)

rebase

actual fix

rebase

fix

rebase

remove TValue typevar as suggested by vlorentz

rebase and update copyrights

Extract the type annotation part of this diff in a dedicated one

Add type annotation in swh.journal.writer and rename the 'stream' arg as 'file_like'

rebase

(not a surprise but) it really needs at least one test of the "overlay feature" for each method...

Actually, with a bit more work, it's the entire db/pytest_plugin.py module that can be deprecated.