Merge pull request #18 from mttdffy/bugfix-saml-nil-values

SAML Tokens: Guards against attributes with nil values
kbeckman · May 27, 2015 · 9c91c9f · 9c91c9f
2 parents 3d41e54 + d327d64
commit 9c91c9f
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/omniauth/strategies/wsfed/saml_1_token.rb b/lib/omniauth/strategies/wsfed/saml_1_token.rb
@@ -31,7 +31,7 @@ def claims
                 value = []
                 attr_element.elements.each { |element| value << element.text }
               else
-                value = attr_element.elements.first.text.lstrip.rstrip
+                value = attr_element.elements.first.text.to_s.lstrip.rstrip
               end
 
               result[name] = value

diff --git a/lib/omniauth/strategies/wsfed/saml_2_token.rb b/lib/omniauth/strategies/wsfed/saml_2_token.rb
@@ -31,7 +31,7 @@ def claims
                 value = []
                 attr_element.elements.each { |element| value << element.text }
               else
-                value = attr_element.elements.first.text.lstrip.rstrip
+                value = attr_element.elements.first.text.to_s.lstrip.rstrip
               end
 
               result[name] = value