openstack
/
keystoneauth
Code Issues Proposed changes

Merge "mark a few oidc parameters as required"

This commit is contained in:
Jenkins 2016-11-09 17:13:29 +00:00 committed by Gerrit Code Review
parent 94eb71c61e 827895281b
commit 42a6bf8dfc
2 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
keystoneauth1/loading/_plugins/identity/v3.py
View File

@ -148,8 +148,9 @@ class OpenIDConnectPassword(_OpenIDConnectBase):
options = super(OpenIDConnectPassword, self).get_options()
options.extend([
loading.Opt('username', help='Username'),
loading.Opt('password', secret=True, help='Password'),
loading.Opt('username', help='Username', required=True),
loading.Opt('password', secret=True,
help='Password', required=True),
])
return options
@ -166,7 +167,8 @@ class OpenIDConnectAuthorizationCode(_OpenIDConnectBase):
options.extend([
loading.Opt('redirect-uri', help='OpenID Connect Redirect URL'),
loading.Opt('authorization-code', secret=True,
loading.Opt('code', secret=True, required=True,
deprecated=[loading.Opt('authorization-code')],
help='OAuth 2.0 Authorization Code'),
])
@ -183,7 +185,7 @@ class OpenIDConnectAccessToken(loading.BaseFederationLoader):
options = super(OpenIDConnectAccessToken, self).get_options()
options.extend([
loading.Opt('access-token', secret=True,
loading.Opt('access-token', secret=True, required=True,
help='OAuth 2.0 Access Token'),
])
return options

2
keystoneauth1/tests/unit/loading/test_v3.py
View File

@ -227,7 +227,7 @@ class OpenIDConnectAuthCodeTests(OpenIDConnectBaseTests, utils.TestCase):
def test_options(self):
options = loading.get_plugin_loader(self.plugin_name).get_options()
self.assertTrue(
set(['redirect-uri', 'authorization-code']).issubset(
set(['redirect-uri', 'code']).issubset(
set([o.name for o in options]))
)