Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Packing SAML Response attribute values in an array when there is more than one xml value within a node #117

Closed
coder-cooke opened this issue Mar 14, 2014 · 1 comment
Closed

Packing SAML Response attribute values in an array when there is more than one xml value within a node #117

coder-cooke opened this issue Mar 14, 2014 · 1 comment

Comments

@coder-cooke
Copy link

We would like to return the attribute values as an array where there is more than one field in an array.

This is similar to #112 but is not identical. That request is dealing with multiple nodes with the same name, whereas this is one node with multiple values.
coder-cooke pushed a commit to coupa-ops/ruby-saml that referenced this issue Mar 14, 2014
Fix for ruby-saml issue SAML-Toolkits#117
ab305d5
borgand pushed a commit to borgand/ruby-saml that referenced this issue Apr 1, 2014
@borgand
Fix for ruby-saml issue SAML-Toolkits#117
7021249
borgand added a commit to borgand/ruby-saml that referenced this issue Apr 1, 2014
@borgand
Testcase for issue SAML-Toolkits#117: extract multiple values from SA…
7e52505 
…ML Response
borgand pushed a commit to borgand/ruby-saml that referenced this issue Apr 1, 2014
@borgand
Fix for ruby-saml issue SAML-Toolkits#117
a6c0454
borgand added a commit to borgand/ruby-saml that referenced this issue Apr 1, 2014
@borgand
Testcase for issue SAML-Toolkits#117: extract multiple values from SA…
8a0821b 
…ML Response
borgand referenced this issue in borgand/ruby-saml Apr 1, 2014
@borgand
Add class AttributeValue that fixes onelogin#117
446720c 
AttributeValue is a String, but `AttributeValue#values` will return
all values for that attribute.
@borgand borgand mentioned this issue Apr 1, 2014
Merged
@borgand
Copy link
Contributor

borgand commented Apr 1, 2014

I created a pull request #122 that fixes this issue in a backwards compatible manner

Lordnibbler pushed a commit that referenced this issue Apr 1, 2014
Merge pull request #122 from borgand/multiple-attribute-values
3e7a9a9 
Fixes #112 and #117 in a backwards compatible manner
kwugirl referenced this issue in newrelic/ruby-saml Jun 27, 2014
@borgand @kwugirl
Add class AttributeValue that fixes onelogin#117
a52511e 
AttributeValue is a String, but `AttributeValue#values` will return
all values for that attribute.
@pitbulk pitbulk mentioned this issue Sep 9, 2014
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@borgand @coder-cooke