From bded1a8c1e6c0d2db30da2e1b0f04e1f04cb493f Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <tkajinam@redhat.com>
Date: Mon, 12 Oct 2020 14:02:49 +0900
Subject: [PATCH] Add support for the keystone_authtoken/service_type parameter

Change-Id: Ica2918b3235f22348598507ec84b264e627322db
---
 manifests/keystone/authtoken.pp                            | 7 +++++++
 .../keystone-authtoken-service_type-1a6b624c522108b6.yaml  | 5 +++++
 spec/classes/barbican_keystone_authtoken_spec.rb           | 3 +++
 3 files changed, 15 insertions(+)
 create mode 100644 releasenotes/notes/keystone-authtoken-service_type-1a6b624c522108b6.yaml

diff --git a/manifests/keystone/authtoken.pp b/manifests/keystone/authtoken.pp
index 596d185b..0d76ba52 100644
--- a/manifests/keystone/authtoken.pp
+++ b/manifests/keystone/authtoken.pp
@@ -177,6 +177,11 @@
 #   true/false
 #   Defaults to $::os_service_default.
 #
+# [*service_type*]
+#  (Optional) The name or type of the service as it appears in the service
+#  catalog. This is used to validate tokens that have restricted access rules.
+#  Defaults to $::os_service_default.
+#
 # [*interface*]
 #  (Optional) Interface to use for the Identity API endpoint. Valid values are
 #  "public", "internal" or "admin".
@@ -217,6 +222,7 @@ class barbican::keystone::authtoken(
   $token_cache_time               = $::os_service_default,
   $service_token_roles            = $::os_service_default,
   $service_token_roles_required   = $::os_service_default,
+  $service_type                   = $::os_service_default,
   $interface                      = $::os_service_default,
 ) {
 
@@ -261,6 +267,7 @@ class barbican::keystone::authtoken(
     token_cache_time               => $token_cache_time,
     service_token_roles            => $service_token_roles,
     service_token_roles_required   => $service_token_roles_required,
+    service_type                   => $service_type,
     interface                      => $interface,
   }
 }
diff --git a/releasenotes/notes/keystone-authtoken-service_type-1a6b624c522108b6.yaml b/releasenotes/notes/keystone-authtoken-service_type-1a6b624c522108b6.yaml
new file mode 100644
index 00000000..9b2b26ad
--- /dev/null
+++ b/releasenotes/notes/keystone-authtoken-service_type-1a6b624c522108b6.yaml
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    The new ``barbican::keystone::authtoken::service_type`` parameter has been
+    added to configure the service_type parameter in authtoken middleware.
diff --git a/spec/classes/barbican_keystone_authtoken_spec.rb b/spec/classes/barbican_keystone_authtoken_spec.rb
index 26186a68..e5417ca2 100644
--- a/spec/classes/barbican_keystone_authtoken_spec.rb
+++ b/spec/classes/barbican_keystone_authtoken_spec.rb
@@ -63,6 +63,7 @@ describe 'barbican::keystone::authtoken' do
         is_expected.to contain_barbican_config('keystone_authtoken/token_cache_time').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_barbican_config('keystone_authtoken/service_token_roles').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_barbican_config('keystone_authtoken/service_token_roles_required').with_value('<SERVICE DEFAULT>')
+        is_expected.to contain_barbican_config('keystone_authtoken/service_type').with_value('<SERVICE DEFAULT>')
         is_expected.to contain_barbican_config('keystone_authtoken/interface').with_value('<SERVICE DEFAULT>')
       end
     end
@@ -104,6 +105,7 @@ describe 'barbican::keystone::authtoken' do
           :token_cache_time                     => '301',
           :service_token_roles                  => ['service'],
           :service_token_roles_required         => false,
+          :service_type                         => 'dentity',
           :interface                            => 'internal',
         })
       end
@@ -142,6 +144,7 @@ describe 'barbican::keystone::authtoken' do
         is_expected.to contain_barbican_config('keystone_authtoken/token_cache_time').with_value(params[:token_cache_time])
         is_expected.to contain_barbican_config('keystone_authtoken/service_token_roles').with_value(params[:service_token_roles])
         is_expected.to contain_barbican_config('keystone_authtoken/service_token_roles_required').with_value(params[:service_token_roles_required])
+        is_expected.to contain_barbican_config('keystone_authtoken/service_type').with_value(params[:service_type])
         is_expected.to contain_barbican_config('keystone_authtoken/interface').with_value(params[:interface])
       end