gitlab.lister: Remove request_params method override

This should have been removed along with the code in b8162127.

The request authentication has been reworked so that all listers use the same
credentials dict.

Related b8162127
Related T1772

swh/lister/gitlab/lister.py

@@ -2,7 +2,6 @@
 # License: GNU General Public License version 3, or any later version
 # See top-level LICENSE file for more information
 
-import random
 import time
 from urllib3.util import parse_url
 
@@ -28,34 +27,6 @@ class GitLabLister(PageByPageHttpLister):
             self.PATH_TEMPLATE = '%s&per_page=%s' % (
                 self.PATH_TEMPLATE, per_page)
 
-    def request_params(self, identifier):
-        """Get the full parameters passed to requests given the
-        transport_request identifier.
-
-        For the gitlab lister, the 'credentials' entries is configured
-        per instance. For example::
-
-          - credentials:
-            - gitlab.com:
-              - username: user0
-                password: <pass>
-              - username: user1
-                password: <pass>
-              - ...
-            - other-gitlab-instance:
-              ...
-
-        """
-        params = {
-            'headers': self.request_headers() or {}
-        }
-        creds_lister = self.config['credentials'].get(self.instance)
-        if creds_lister:
-            auth = random.choice(creds_lister)
-            if auth:
-                params['auth'] = (auth['username'], auth['password'])
-        return params
-
     def uid(self, repo):
         return '%s/%s' % (self.instance, repo['path_with_namespace'])
 

[Phabricator Migration user]

Some references in the commit message have been migrated:

• T1772 is now swh/infra/puppet/puppet-swh-site#1772 (closed)