Skip to content

Don't parse html with xml parser #53

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

Closed
liquidpele wants to merge 5 commits into from
Closed

Don't parse html with xml parser #53

Changes from 1 commit
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
b60149c
Don't parse html with xml parser, and instead get the data we need di…
Apr 11, 2017
d732a3b
Don't use the private method
Apr 11, 2017
43d182e
Remove un-used import, utilize the default option of getattr
Apr 11, 2017
501095a
Merge in my other PR about failing back to supported idp bindings, an…
Apr 12, 2017
2868b31
Somehow deleted the get_location method?
Apr 12, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Somehow deleted the get_location method?
  • Loading branch information
Reece authored and Reece committed Apr 12, 2017
commit 2868b317dfacbac4a335b952975dff7e5d806454
11 changes: 11 additions & 0 deletions djangosaml2/utils.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -48,3 +48,14 @@ def get_idp_sso_supported_bindings(idp_entity_id=None):
return meta.service(idp_entity_id, 'idpsso_descriptor', 'single_sign_on_service').keys()
except UnknownSystemEntity:
return []


def get_location(http_info):
"""Extract the redirect URL from a pysaml2 http_info object"""
assert 'headers' in http_info
headers = http_info['headers']

assert len(headers) == 1
header_name, header_value = headers[0]
assert header_name == 'Location'
return header_value