Merge pull request #20 from filintod/master

fix acl role with policy
poppyred · Nov 30, 2020 · b105755 · b105755
2 parents c2596c6 + 9cbc941
commit b105755
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/consul/base.py b/consul/base.py
@@ -1074,17 +1074,18 @@ def list(
                     :return:
                 """
 
+                path = '/v1/acl/roles'
+                params = []
                 if policy:
-                    path = '/v1/acl/roles?policy=%s' % policy
-                else:
-                    path = '/v1/acl/roles'
+                    params.append(('policy', policy))
                 headers = {}
                 token = token or self.agent.token
                 if token:
                     headers['X-Consul-Token'] = token
 
                 return self.agent.http.get(CB.json(),
                                            path=path,
+                                           params=params,
                                            headers=headers)
 
         class AuthMethod(object):