Skip to content

Commit 905df8b

Browse files
oneswigmarkgoddard
authored andcommitted
OpenID Connect certifiate file is optional
Some ID provider configurations do not require a certificate file. Change the logic to allow this, and update documentation accordingly. Change-Id: I2c34a6b5894402bbebeb3fb96768789bc3c7fe84 (cherry picked from commit 78f29fd)
1 parent 9ef2526 commit 905df8b

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

ansible/roles/keystone/tasks/config-federation-oidc.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,6 +52,7 @@
5252
with_items: "{{ keystone_identity_providers }}"
5353
when:
5454
- item.protocol == 'openid'
55+
- item.certificate_file is defined
5556
- inventory_hostname in groups[keystone.group]
5657

5758
- name: Copying OpenStack Identity Providers attribute mappings

doc/source/reference/shared-services/keystone-guide.rst

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -247,8 +247,8 @@ Identity provider's endpoint:
247247
certificate_file
248248
****************
249249

250-
Path to the Identity Provider certificate file, the file must be named as
251-
'certificate-key-id.pem'. E.g.
250+
Optional path to the Identity Provider certificate file. If included,
251+
the file must be named as 'certificate-key-id.pem'. E.g.:
252252

253253
.. code-block::
254254

0 commit comments

Comments
 (0)