HomeSoftware Heritage

phabricator.lister: Use credentials setup from configuration file

This commit no longer exists in the repository. It may have been part of a branch which was deleted.

Description

phabricator.lister: Use credentials setup from configuration file

Prior to this commit, this expected the api.token to be provided at task
initialization. That behavior has been kept for cli purposes. It's no good for
production purposes though (as this leaks the credentials in the scheduler db).

So now, the credentials is fetched from the lister's configuration file as the
other listers do.

Another change is the authentication mechanism which is slighly different. It's
not using a basic auth mechanism. It's expecting an api.token query
parameter so the request_params is overriden to provide that.

Related T1809

Details

Provenance
ardumontAuthored on Jun 14 2019, 6:10 PM
ardumontPushed on Jun 17 2019, 5:48 PM
Differential Revision
D1588: phabricator.lister: Use credentials setup from config file
Build Status
Buildable 6263
Build 8663: test-and-build

Commit No Longer Exists

This commit no longer exists in the repository.