Skip to content

Commit 4204afc

Browse files
committed
same code as authnrequest for logoutrequest
1 parent b0301c9 commit 4204afc

File tree

1 file changed

+11
-2
lines changed

1 file changed

+11
-2
lines changed

lib/onelogin/ruby-saml/logoutrequest.rb

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -89,6 +89,11 @@ def create_params(settings, params={})
8989
# @return [String] The SAMLRequest String.
9090
#
9191
def create_logout_request_xml_doc(settings)
92+
document = create_xml_document(settings)
93+
sign_document(document, settings)
94+
end
95+
96+
def create_xml_document(settings)
9297
time = Time.now.utc.strftime("%Y-%m-%dT%H:%M:%SZ")
9398

9499
request_doc = XMLSecurity::Document.new
@@ -122,14 +127,18 @@ def create_logout_request_xml_doc(settings)
122127
sessionindex.text = settings.sessionindex
123128
end
124129

130+
request_doc
131+
end
132+
133+
def sign_document(document, settings)
125134
# embed signature
126135
if settings.security[:logout_requests_signed] && settings.private_key && settings.certificate && settings.security[:embed_sign]
127136
private_key = settings.get_sp_key
128137
cert = settings.get_sp_cert
129-
request_doc.sign_document(private_key, cert, settings.security[:signature_method], settings.security[:digest_method])
138+
document.sign_document(private_key, cert, settings.security[:signature_method], settings.security[:digest_method])
130139
end
131140

132-
request_doc
141+
document
133142
end
134143
end
135144
end

0 commit comments

Comments
 (0)