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

Extract all bindings and support multiple certificates when decoding idp metadata #26

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

Extract all bindings and support multiple certificates when decoding idp metadata #26

wants to merge 2 commits into from

Commits on Jun 29, 2016

  1. extract all bindings in esaml:decode_idp_metadata/1 

    Before, this would only look for HTTP-POST login locations. Now, it will
extract any location it finds: post, redirect, or artifact. The way this
is achieved is not the most beautiful, but allows for easy matching when
`#esaml_idp_metadata` is used.

Also adds okta-ish test metadata.
    @srenatus
    srenatus authored and kmacgugan committed Jun 29, 2016
    Configuration menu
    Copy the full SHA
    c69bc57 View commit details
    Browse the repository at this point in the history

  2. extract multiple certificates from IdP metadata 

    Changes the `esaml_idp_metadata` record: `certificate` -> `certificates`,
now of type `[binary()]` instead of `binary()`, with `undefined` instead
of `[]` winning if there's nothing in the metadata xml document.

Test data included (Azure AD).
    @srenatus
    srenatus authored and kmacgugan committed Jun 29, 2016
    Configuration menu
    Copy the full SHA
    eddded6 View commit details
    Browse the repository at this point in the history