feat: Add issuer information to the metadata endpoint, to allow IdPs …

…to properly configure themselves.
omniauth · Sep 16, 2016 · 9f56c23 · 9f56c23
1 parent f6f9b97
commit 9f56c23
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/lib/omniauth/strategies/saml.rb b/lib/omniauth/strategies/saml.rb
@@ -115,6 +115,7 @@ def other_phase
           settings = OneLogin::RubySaml::Settings.new(options)
           if options.request_attributes.length > 0
             settings.attribute_consuming_service.service_name options.attribute_service_name
+            settings.issuer = options.issuer
             options.request_attributes.each do |attribute|
               settings.attribute_consuming_service.add_attribute attribute
             end

diff --git a/spec/omniauth/strategies/saml_spec.rb b/spec/omniauth/strategies/saml_spec.rb
@@ -207,6 +207,7 @@ def post_xml(xml=:example_response)
 
   describe 'GET /auth/saml/metadata' do
     before do
+      saml_options[:issuer] = 'http://example.com/SAML'
       get '/auth/saml/metadata'
     end
 
@@ -220,6 +221,8 @@ def post_xml(xml=:example_response)
       last_response.body.should match /first_name/
       last_response.body.should match /last_name/
       last_response.body.should match /Required attributes/
+      last_response.body.should match /entityID/
+      last_response.body.should match /http:\/\/example.com\/SAML/
     end
   end