saml-kit 0.2.7 → 0.2.8

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: '029d3a033d81aac8add1be06a783516f239b7b38cec4bb6e701e010c80b82e98'
-  data.tar.gz: a29f4fd46aa450d8392376cdb38fe2510575b7a51e1217987719f1828c1defbc
+  metadata.gz: c496796275792465ee1906c67364da02c7633058aa58ba06cd8f3471f24aadf7
+  data.tar.gz: '090aef04081e81bad6805fecbc73daeaa88fdeaf2d88dbcaeb42c8e204416e0f'
 SHA512:
-  metadata.gz: 694fe63f05d9b011d998f4c144faf75f9b852e63ad01badf522b23e654038c5d6655334a24b9c366d5920df0f01772fbfdf0190c5946cbca79a007b1a74eb64c
-  data.tar.gz: 4b497ec560bbd279cc957ee2765bd1e6f1ea48a591e54a58f434522984bcf0ebcf210e343b8a02ecc0e268d6d95221cd21138fb2b655b008203f0520cc8cebb4
+  metadata.gz: f1c3064dfb1e0ae83c665d6e43da37ad639b384491b1100d5c3cdf4b52a6f18e3cf169cb81e4b93516dffc4b73b3f3318c99d6e43b4fb5684c51503917437ff4
+  data.tar.gz: '020684ed30855042cf53c804ec1e12355ffe1ac032365abc94de85b757c8e330418435cde36cc8e081d231a7cccbf8777012adfff44e4003d02d26b14eaecf23'

data/lib/saml/kit/bindings/binding.rb

@@ -48,10 +48,14 @@ module Saml
         protected
 
         def saml_param_from(params)
-          if params['SAMLRequest'].present?
-            params['SAMLRequest']
-          elsif params['SAMLResponse'].present?
-            params['SAMLResponse']
+          parameters = {
+            SAMLRequest: params[:SAMLRequest] || params['SAMLRequest'],
+            SAMLResponse: params[:SAMLResponse] || params['SAMLResponse'],
+          }
+          if parameters[:SAMLRequest].present?
+            parameters[:SAMLRequest]
+          elsif parameters[:SAMLResponse].present?
+            parameters[:SAMLResponse]
           else
             raise ArgumentError.new("SAMLRequest or SAMLResponse parameter is required.")
           end

data/lib/saml/kit/bindings/url_builder.rb

@@ -10,10 +10,14 @@ module Saml
         end
 
         def build(saml_document, relay_state: nil)
-          payload = canonicalize(saml_document, relay_state)
           if configuration.sign?
+            payload = canonicalize(saml_document, relay_state)
             "#{saml_document.destination}?#{payload}&Signature=#{signature_for(payload)}"
           else
+            payload = to_query_string(
+              saml_document.query_string_parameter => serialize(saml_document.to_xml),
+              'RelayState' => relay_state,
+            )
             "#{saml_document.destination}?#{payload}"
           end
         end
@@ -26,11 +30,15 @@ module Saml
         end
 
         def canonicalize(saml_document, relay_state)
-          {
+          to_query_string(
             saml_document.query_string_parameter => serialize(saml_document.to_xml),
             'RelayState' => relay_state,
             'SigAlg' => Saml::Kit::Namespaces::SHA256,
-          }.map do |(key, value)|
+          )
+        end
+
+        def to_query_string(query_params)
+          query_params.map do |(key, value)|
             value.present? ? "#{key}=#{escape(value)}" : nil
           end.compact.join('&')
         end

data/lib/saml/kit/configuration.rb

@@ -17,7 +17,7 @@ module Saml
       end
 
       def add_key_pair(certificate, private_key, password:, use: :signing)
-        @key_pairs.push(KeyPair.new(certificate, private_key, password, use))
+        @key_pairs.push(KeyPair.new(certificate, private_key, password, use.to_sym))
       end
 
       def generate_key_pair_for(use:, password: SecureRandom.uuid)

data/lib/saml/kit/identity_provider_metadata.rb

@@ -29,8 +29,8 @@ module Saml
         end
       end
 
-      def login_request_for(binding:, relay_state: nil)
-        builder = Saml::Kit::AuthenticationRequest.builder do |x|
+      def login_request_for(binding:, relay_state: nil, configuration: Saml::Kit.configuration)
+        builder = Saml::Kit::AuthenticationRequest.builder(configuration: configuration) do |x|
           x.embed_signature = want_authn_requests_signed
           yield x if block_given?
         end

data/lib/saml/kit/key_pair.rb

@@ -1,7 +1,7 @@
 module Saml
   module Kit
     class KeyPair
-      attr_reader :certificate, :private_key
+      attr_reader :certificate, :private_key, :use
 
       def initialize(certificate, private_key, password, use)
         @use = use

data/lib/saml/kit/version.rb

@@ -1,5 +1,5 @@
 module Saml
   module Kit
-    VERSION = "0.2.7"
+    VERSION = "0.2.8"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: saml-kit
 version: !ruby/object:Gem::Version
-  version: 0.2.7
+  version: 0.2.8
 platform: ruby
 authors:
 - mo khan
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2017-12-16 00:00:00.000000000 Z
+date: 2017-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activemodel