ruby-samlnechotech 0.7.31 → 0.7.32

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3bf49be077e9d389a9fa0dc4952d2197d5983149
-  data.tar.gz: 83da3b834385465d0a1a3cd68ae3f23c9bc1f093
+  metadata.gz: 2bb37f4e246d9b31c8bfe72acf8731e8c6ac55ea
+  data.tar.gz: 6c08d064ce2b17103ef948a626e4e143debdbdf9
 SHA512:
-  metadata.gz: f8c0ef6658fd72a46c1282b04f57b90477062e74b4c8a93436bd0d7b5d4f208525837b2012f850fffb8d38cda2aa8ab46d8cc90bf586dcde5d22bc8137a8c639
-  data.tar.gz: 0484b14583a4c3f7e8ed0f9cd952c8908a4f34edd2bbfa74b3bd05092a838ae0ddd4631b107a18def478a56af7fd5ff9ad2beba34bda4b210cea9dc04d978ab0
+  metadata.gz: 1e0b8c88e5f78181a49990abffca12800aa7bbeac02207a5127172b086c5a96cca0c507d6c4a1fb50254fd266c1d42335f46728376f11c3ae6a7097b266bad77
+  data.tar.gz: b09a66df1703f9cdd6e8397c32f5625428e7b9e9f27da59589e3e2e6f5fa0c4da9b520906aa488a52b73257553eee4ec13f1f5e6ef07543024475cc426f63ce9

data/lib/onelogin/ruby-samlnechotech/authrequest.rb

@@ -18,7 +18,7 @@ module Onelogin
         request = ""
         request_doc.write(request)
 
-        Logging.debug "Created AuthnRequest: #{request}"
+        Logging.debug "Created AuthnRequest:\n#{request}\n"
 
         request           = Zlib::Deflate.deflate(request, 9)[2..-5] if settings.compress_request
         base64_request    = Base64.encode64(request)
@@ -36,7 +36,7 @@ module Onelogin
       def create_authentication_xml_doc(settings)
         uuid = "_" + UUID.new.generate
         time = Time.now.utc.strftime("%Y-%m-%dT%H:%M:%SZ")
-        # Create AuthnRequest root element using REXML 
+        # Create AuthnRequest root element using REXML
         request_doc = REXML::Document.new
 
         root = request_doc.add_element "samlp:AuthnRequest", { "xmlns:samlp" => "urn:oasis:names:tc:SAML:2.0:protocol" }
@@ -45,7 +45,7 @@ module Onelogin
         root.attributes['Version'] = "2.0"
         root.attributes['Destination'] = settings.idp_sso_target_url unless settings.idp_sso_target_url.nil?
         root.attributes['IsPassive'] = settings.passive unless settings.passive.nil?
-        root.attributes['ForceAuthN'] = "#{settings.force_authn}" unless settings.force_authn.nil?
+        root.attributes['ForceAuthn'] = settings.force_authn unless settings.force_authn.nil?
 
         # Conditionally defined elements based on settings
         if settings.assertion_consumer_service_url != nil
@@ -56,7 +56,7 @@ module Onelogin
           issuer.text = settings.issuer
         end
         if settings.name_identifier_format != nil
-          root.add_element "samlp:NameIDPolicy", { 
+          root.add_element "samlp:NameIDPolicy", {
               "xmlns:samlp" => "urn:oasis:names:tc:SAML:2.0:protocol",
               # Might want to make AllowCreate a setting?
               "AllowCreate" => "true",
@@ -65,14 +65,14 @@ module Onelogin
         end
 
         # BUG fix here -- if an authn_context is defined, add the tags with an "exact"
-        # match required for authentication to succeed.  If this is not defined, 
+        # match required for authentication to succeed.  If this is not defined,
         # the IdP will choose default rules for authentication.  (Shibboleth IdP)
         if settings.authn_context != nil
-          requested_context = root.add_element "samlp:RequestedAuthnContext", { 
+          requested_context = root.add_element "samlp:RequestedAuthnContext", {
             "xmlns:samlp" => "urn:oasis:names:tc:SAML:2.0:protocol",
             "Comparison" => "exact",
           }
-          class_ref = requested_context.add_element "saml:AuthnContextClassRef", { 
+          class_ref = requested_context.add_element "saml:AuthnContextClassRef", {
             "xmlns:saml" => "urn:oasis:names:tc:SAML:2.0:assertion",
           }
           class_ref.text = settings.authn_context

data/lib/onelogin/ruby-samlnechotech/metadata.rb

@@ -3,7 +3,7 @@ require "rexml/xpath"
 require "uri"
 
 # Class to return SP metadata based on the settings requested.
-# Return this XML in a controller, then give that URL to the the 
+# Return this XML in a controller, then give that URL to the the
 # IdP administrator.  The IdP will poll the URL and your settings
 # will be updated automatically
 module Onelogin
@@ -57,7 +57,7 @@ module Onelogin
         # pretty print the XML so IdP administrators can easily see what the SP supports
         meta_doc.write(ret, 1)
 
-        Logging.debug "Generated metadata:\n#{ret}"
+        Logging.debug "Generated metadata:\n#{ret}\n"
 
         ret
       end

data/lib/onelogin/ruby-samlnechotech/response.rb

@@ -35,15 +35,14 @@ module Onelogin
         base64_cert = document.elements["//ds:X509Certificate"].text
         cert_text = Base64.decode64(base64_cert)
         cert = OpenSSL::X509::Certificate.new(cert_text)
-        r_logger = Rails.logger rescue nil
         # check cert matches registered idp cert
         fingerprint = Digest::SHA1.hexdigest(cert.to_der)
-        r_logger.info("fingerprint = " + fingerprint) if !r_logger.nil?
+        Logging.debug "Fingerprint:\n#{fingerprint}\n"
         valid_flag = fingerprint == idp_cert_fingerprint.gsub(":", "").downcase
 
         return valid_flag if !valid_flag
 
-        document.validate_doc(base64_cert, r_logger)
+        document.validate_doc(base64_cert, Logging)
       end
 
       def validate!
@@ -99,6 +98,7 @@ module Onelogin
       # Checks the status of the response for a "Success" code
       # (nechotech: ...or a "NoPassive" secondary status code)
       def success?
+        log()
         @status_code ||= begin
           node = REXML::XPath.first(document, "/p:Response/p:Status/p:StatusCode", { "p" => PROTOCOL, "a" => ASSERTION })
           primary_status = node.attributes["Value"]
@@ -127,6 +127,10 @@ module Onelogin
         end
       end
 
+      def log
+        Logging.debug "SAML Response:\n#{document}\n"
+      end
+
       private
 
       def validation_error(message)

data/lib/onelogin/ruby-samlnechotech/settings.rb

@@ -22,7 +22,7 @@ module Onelogin
       attr_accessor :idp_isp_cert
 
       private
-      
+
       DEFAULTS = {:compress_request => true, :double_quote_xml_attribute_values => false}
     end
   end

data/lib/onelogin/ruby-samlnechotech/version.rb

@@ -1,5 +1,5 @@
 module Onelogin
   module Saml
-    VERSION = '0.7.31'
+    VERSION = '0.7.32'
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby-samlnechotech
 version: !ruby/object:Gem::Version
-  version: 0.7.31
+  version: 0.7.32
 platform: ruby
 authors:
 - OneLogin LLC, beekermememe
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-05-29 00:00:00.000000000 Z
+date: 2013-06-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: canonix