diff --git a/swh/lister/phabricator/lister.py b/swh/lister/phabricator/lister.py --- a/swh/lister/phabricator/lister.py +++ b/swh/lister/phabricator/lister.py @@ -18,13 +18,11 @@ MODEL = PhabricatorModel LISTER_NAME = 'phabricator' - def __init__(self, forge_url, instance=None, api_token=None, - override_config=None): + def __init__(self, forge_url, instance=None, override_config=None): if forge_url.endswith("/"): forge_url = forge_url[:-1] self.forge_url = forge_url api_baseurl = '%s/api/diffusion.repository.search' % forge_url - self.api_token = api_token if not instance: instance = urllib.parse.urlparse(forge_url).hostname self.instance = instance @@ -63,8 +61,6 @@ """ params = {} params['headers'] = self.request_headers() or {} - if self.api_token: - return self._build_query_params(params, self.api_token) instance_creds = self.request_instance_credentials() if not instance_creds: raise ValueError( diff --git a/swh/lister/phabricator/tests/test_lister.py b/swh/lister/phabricator/tests/test_lister.py --- a/swh/lister/phabricator/tests/test_lister.py +++ b/swh/lister/phabricator/tests/test_lister.py @@ -28,9 +28,14 @@ """ if override_config or self.fl is None: - self.fl = self.Lister(forge_url='https://fakeurl', instance='fake', - api_token='a-1', - override_config=override_config) + credentials = {'phabricator': {'fake': [ + {'password': 'toto'} + ]}} + override_config = dict(credentials=credentials, + **(override_config or {})) + self.fl = self.Lister( + forge_url='https://fakeurl', instance='fake', + override_config=override_config) self.fl.INITIAL_BACKOFF = 1 self.fl.reset_backoff()