summaryrefslogtreecommitdiff
path: root/lib/gitlab/saml
diff options
context:
space:
mode:
authorPatricio Cano <suprnova32@gmail.com>2016-04-06 16:03:35 -0500
committerPatricio Cano <suprnova32@gmail.com>2016-04-06 16:03:35 -0500
commit3a36fa895724aedb4bd919ec91cc00a24415e712 (patch)
tree9ea8cc8a2d65cc6a0edafc3249275b69aab48033 /lib/gitlab/saml
parent7efaf22bccb16b381f7e76054d084e741006fc5f (diff)
downloadgitlab-ce-3a36fa895724aedb4bd919ec91cc00a24415e712.tar.gz
Fix error that was causing only one group to be returned and corrected specs to use the proper attribute type
Diffstat (limited to 'lib/gitlab/saml')
-rw-r--r--lib/gitlab/saml/auth_hash.rb4
1 files changed, 3 insertions, 1 deletions
diff --git a/lib/gitlab/saml/auth_hash.rb b/lib/gitlab/saml/auth_hash.rb
index 5ffccc0e100..3414d24ca73 100644
--- a/lib/gitlab/saml/auth_hash.rb
+++ b/lib/gitlab/saml/auth_hash.rb
@@ -9,7 +9,9 @@ module Gitlab
private
def get_raw(key)
- auth_hash.extra[:raw_info][key]
+ # Needs to call `all` because of https://github.com/onelogin/ruby-saml/blob/master/lib/onelogin/ruby-saml/attributes.rb#L78
+ # otherwise just the first value is returned
+ auth_hash.extra[:raw_info].all[key]
end
end