Skip to content

Commit dd85491

Browse files
committed
Rename idp_cert_fingerprint_alg var by idp_cert_fingerprint_algorithm
1 parent a7e193a commit dd85491

File tree

3 files changed

+5
-5
lines changed

3 files changed

+5
-5
lines changed

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -116,7 +116,7 @@ def saml_settings
116116
settings.idp_sso_target_url = "https://app.onelogin.com/trust/saml2/http-post/sso/#{OneLoginAppId}"
117117
settings.idp_slo_target_url = "https://app.onelogin.com/trust/saml2/http-redirect/slo/#{OneLoginAppId}"
118118
settings.idp_cert_fingerprint = OneLoginAppCertFingerPrint
119-
settings.idp_cert_fingerprint_alg = "http://www.w3.org/2000/09/xmldsig#sha1"
119+
settings.idp_cert_fingerprint_algorithm = "http://www.w3.org/2000/09/xmldsig#sha1"
120120
settings.name_identifier_format = "urn:oasis:names:tc:SAML:1.1:nameid-format:emailAddress"
121121
122122
# Optional for most SAML IdPs

lib/onelogin/ruby-saml/response.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -140,7 +140,7 @@ def validate(soft = true)
140140
validate_response_state(soft) &&
141141
validate_conditions(soft) &&
142142
validate_issuer(soft) &&
143-
document.validate_document(get_fingerprint, soft, :fingerprint_alg => settings.idp_cert_fingerprint_alg) &&
143+
document.validate_document(get_fingerprint, soft, :fingerprint_alg => settings.idp_cert_fingerprint_algorithm) &&
144144
validate_success_status(soft)
145145
end
146146

@@ -203,7 +203,7 @@ def xpath_first_from_signed_assertion(subelt=nil)
203203
def get_fingerprint
204204
if settings.idp_cert
205205
cert = OpenSSL::X509::Certificate.new(settings.idp_cert)
206-
fingerprint_alg = XMLSecurity::BaseDocument.new.algorithm(settings.idp_cert_fingerprint_alg).new
206+
fingerprint_alg = XMLSecurity::BaseDocument.new.algorithm(settings.idp_cert_fingerprint_algorithm).new
207207
fingerprint_alg.hexdigest(cert.to_der).upcase.scan(/../).join(":")
208208
else
209209
settings.idp_cert_fingerprint

lib/onelogin/ruby-saml/settings.rb

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def initialize(overrides = {})
2323
attr_accessor :idp_slo_target_url
2424
attr_accessor :idp_cert
2525
attr_accessor :idp_cert_fingerprint
26-
attr_accessor :idp_cert_fingerprint_alg
26+
attr_accessor :idp_cert_fingerprint_algorithm
2727
# SP Data
2828
attr_accessor :issuer
2929
attr_accessor :assertion_consumer_service_url
@@ -107,7 +107,7 @@ def get_sp_key
107107
DEFAULTS = {
108108
:assertion_consumer_service_binding => "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-POST".freeze,
109109
:single_logout_service_binding => "urn:oasis:names:tc:SAML:2.0:bindings:HTTP-Redirect".freeze,
110-
:idp_cert_fingerprint_alg => XMLSecurity::Document::SHA1,
110+
:idp_cert_fingerprint_algorithm => XMLSecurity::Document::SHA1,
111111
:compress_request => true,
112112
:compress_response => true,
113113
:security => {

0 commit comments

Comments
 (0)