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.

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

Enabled Multi Valued Attributes and Allow Data url redirection (307) (Optional in config) #95

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

akki-ng
Copy link

@akki-ng akki-ng commented Apr 19, 2017

Till now multivalued attributes were not received properly in RubyCasClient even though these were being sent as CAS Response. Typical use-case for this behaviour is "memberOf" attribute for LDAP/AD.

All data url(.json, .xml) were given 401 if unauthenticated, A new configuration :redirect_all can be given true in environment.rb to allow all urls(data url) to get self authenticated using basic auth.

To accomplish this we had to use 307 status code rather 302 to maintain initial request httpmethod and request body.

Http clients can use 307 to redirect requests properly, For example post call by Httparty will look like this

response = HTTParty.post(
"My_URL_FOR_DATA_POST",
:body => body.to_json,
:headers => headers,
:maintain_method_across_redirects => true,
<———Must
:resend_on_redirect => true
<———Must
)

@akki-ng
Copy link
Author

akki-ng commented Apr 24, 2017

Is there anything pending on my behalf?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants