Merge pull request #246 from stian-fredrikstad/master

On LogoutResponse, the issuer has to be before the status.

Author: pitbulk

lib/onelogin/ruby-saml/slo_logoutresponse.rb

@@ -104,6 +104,11 @@ def create_logout_response_xml_doc(settings, request_id = nil, logout_message =
         root.attributes['InResponseTo'] = request_id unless request_id.nil?
         root.attributes['Destination'] = settings.idp_slo_target_url unless settings.idp_slo_target_url.nil?
 
+        if settings.issuer != nil
+          issuer = root.add_element "saml:Issuer"
+          issuer.text = settings.issuer
+        end
+        
         # add success message
         status = root.add_element 'samlp:Status'
 
@@ -116,11 +121,6 @@ def create_logout_response_xml_doc(settings, request_id = nil, logout_message =
         status_message = status.add_element 'samlp:StatusMessage'
         status_message.text = logout_message
 
-        if settings.issuer != nil
-          issuer = root.add_element "saml:Issuer"
-          issuer.text = settings.issuer
-        end
-
         # embed signature
         if settings.security[:logout_responses_signed] && settings.private_key && settings.certificate && settings.security[:embed_sign]
           private_key = settings.get_sp_key