need ability to configure with a metadata.xml file (rather than URL)

[stevecrozz]

Google in their infinite wisdom has decided they will not provide public metadata URLs for their customers, and so I find myself wanting to configure this project with a static metadata.xml file that I've downloaded.

Would you accept a pull request to add this feature? I'm thinking of a configuration option named idp_metadata_path which would be the path to a metadata file. It would be a mutually exclusive option with idp_metadata_url. If the file is specified and exists and a URL is not specified, then we would read the file and use it instead of the URL. It would be pretty simple as crewjam/saml/samlsp already understands how to handle both URLs and files on disk.

Either way, thanks for putting this project together!

[dustin-decker]

Sounds good to me. I'll be able to review the PR in less than a week from now. ‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐

…

On April 30, 2018 11:27 PM, Stephen Crosby ***@***.***> wrote: Google in their infinite wisdom has decided they will not provide public metadata URLs for their customers, and so I find myself wanting to configure this project with a static metadata.xml file that I've downloaded. Would you accept a pull request to add this feature? I'm thinking of a configuration option named idp_metadata_path which would be the path to a metadata file. It would be a mutually exclusive option with idp_metadata_url. If the file is specified and exists and a URL is not specified, then we would read the file and use it instead of the URL. It would be pretty simple as crewjam/saml/samlsp already understands how to handle both URLs and files on disk. Either way, thanks for putting this project together! — You are receiving this because you are subscribed to this thread. Reply to this email directly, [view it on GitHub](#2), or [mute the thread](https://github.com/notifications/unsubscribe-auth/AFwIOAycLs0zw2Zqt8IaSeOb7zxu3WVFks5tt-QdgaJpZM4Ttjsl).

[dustin-decker]

Hey @stevecrozz, I noticed you forked the project to make the changes you wanted. I'm glad the project was useful for you. It'd be great if you could make a PR to add the optional TLS termination (i've been wanting to add that for a while) and the metadata path.