HomeSoftware Heritage

phabricator.lister: Use credentials setup from configuration file

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
Parents
rDLSecdce4b0cc85: gitlab.lister: Remove request_params method override
Branches
Unknown
Tags
Unknown
References
tag: v0.0.26
Build Status
Buildable 6263
Build 8663: test-and-build