Fix default mutable arg in k2k.py

Change-Id: I9e8d838315d521874e6f92922a9f488ee7029dbc
This commit is contained in:
Rob Cresswell 2017-07-11 15:33:08 +01:00
parent c15dfe5acd
commit a0af708a49
1 changed files with 4 additions and 1 deletions

View File

@ -28,7 +28,7 @@ __all__ = ['K2KAuthPlugin']
class K2KAuthPlugin(base.BasePlugin):
def get_plugin(self, service_provider=None, auth_url=None, plugins=[],
def get_plugin(self, service_provider=None, auth_url=None, plugins=None,
**kwargs):
"""Authenticate using keystone to keystone federation.
@ -42,6 +42,9 @@ class K2KAuthPlugin(base.BasePlugin):
:returns Keystone2Keystone keystone auth plugin
"""
# Avoid mutable default arg for plugins
plugins = plugins or []
# service_provider being None prevents infinite recursion
if utils.get_keystone_version() < 3 or not service_provider:
return None