spid-es 0.0.37 → 0.0.42

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b485b1feeb1b83e2924dd51120876189dbe5640bca45b8130213203b5790c495
-  data.tar.gz: a60a8b7a7179fbae9b10f5c4ee2fe447605127e37bf8ca4b80945683e2594d2b
+  metadata.gz: 98963a0fb7909e5e96d105a61d7b3a0c10378b39617dbaadb0c5dfdf6ac8e044
+  data.tar.gz: 293a46c2471be93d09a6265d28b053f2fce3271d0380ab3981d435f9268023f2
 SHA512:
-  metadata.gz: 9760018b318d2e972e6e6ad4c3d10c27bf40f44e0387bf8c7e3e6fc7852e646f847c4e1c747bd95aa47f6e8dd88d0a84105eba1781880d203221c4af287796e4
-  data.tar.gz: cd6e83e3eb023f934f1947dbd21e18637f79a9961960c99935ca7f3ea4e1b3ff28bd4cd015737265704eff08e01629b37131e4bab7c3998b53c30a856c8918fa
+  metadata.gz: 55da9133c1d43fa72384e17d612c86c81ee0cb88f536991a272285c11d2f4cca582ce346d7288055a73ab97ceaac37104e6aedababba781a60a2d4902d76d6d3
+  data.tar.gz: 08569eb6ffca8f6fa3e34e31a0c2972fdd049e721ffc7774081c8d98b3283a0034f43ca9baa9da15fcbb38305eadbe68a6a1196f19003aedc4ab0de24c0835e5

data/lib/spid/ruby-saml/authrequest.rb

@@ -30,7 +30,7 @@ module Spid::Saml
       request_doc = Spid::XMLSecurityNew::Document.new
       request_doc.context[:attribute_quote] = :quote
       root = request_doc.add_element "saml2p:AuthnRequest", { "xmlns:saml2p" => "urn:oasis:names:tc:SAML:2.0:protocol", 
-                                                              "xmlns:saml" => "urn:oasis:names:tc:SAML:2.0:assertion"
+                                                              "xmlns:saml2" => "urn:oasis:names:tc:SAML:2.0:assertion"
                                                              }
       root.attributes['ID'] = uuid
       root.attributes['IssueInstant'] = time
@@ -53,9 +53,9 @@ module Spid::Saml
       end
 
       unless @settings.issuer.blank?
-        issuer = root.add_element "saml:Issuer"
-        #tolto NameQualifier, con aggregatore non mettendolo corrispondente all'entity_id non funziona...
-        #issuer.attributes['NameQualifier'] = ( @settings.aggregato ? @settings.sp_name_qualifier : @settings.issuer ) 
+        issuer = root.add_element "saml2:Issuer", { "xmlns:saml2" => "urn:oasis:names:tc:SAML:2.0:assertion" }
+        #l'attributo NameQualifier deve essere presente, non rispetta saml...
+        issuer.attributes['NameQualifier'] = ( @settings.aggregato ? @settings.sp_name_qualifier : @settings.issuer ) 
         issuer.attributes['Format'] = "urn:oasis:names:tc:SAML:2.0:nameid-format:entity"
         issuer.text = @settings.issuer #questo valore deve essere uguale al #entityID dei metadata che usa @settings.issuer
       end
@@ -88,7 +88,7 @@ module Spid::Saml
         }
         context_class = []
         @settings.authn_context.each_with_index{ |context, index|
-          context_class[index] = requested_context.add_element "saml:AuthnContextClassRef"
+          context_class[index] = requested_context.add_element "saml2:AuthnContextClassRef"
           context_class[index].text = context
         }
         

data/lib/spid/ruby-saml/response.rb

@@ -248,7 +248,9 @@ module Spid
             if settings
               idp_metadata = Spid::Saml::Metadata.new(settings).get_idp_metadata
             end
-            
+            #verifico se sono stati scaricati i metadati dell'idp
+            return false if validate_metadata_idp(soft) == false
+
             #carico nei setting l'idp_entity_id  
             entity_descriptor_element = REXML::XPath.first(idp_metadata,"/EntityDescriptor")
             if !entity_descriptor_element.nil? 
@@ -334,6 +336,11 @@ module Spid
         end
 
 
+        #validate presenza dei metadata per idp
+        def validate_metadata_idp(soft=true)
+          return (soft ? false : validation_error("Metadata idp non raggiungibile per #{settings.idp_entity_id}")) if document.blank?
+          true
+        end
 
         # Validates the SAML version (2.0)
         # If fails, the error is added to the errors array.

data/spid-es.gemspec

@@ -2,7 +2,7 @@ $LOAD_PATH.push File.expand_path('../lib', __FILE__)
 
 Gem::Specification.new do |s|
   s.name = 'spid-es'
-  s.version = '0.0.37'
+  s.version = '0.0.42'
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["Fabiano Pavan"]

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spid-es
 version: !ruby/object:Gem::Version
-  version: 0.0.37
+  version: 0.0.42
 platform: ruby
 authors:
 - Fabiano Pavan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-10-21 00:00:00.000000000 Z
+date: 2020-12-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: canonix