Skip to content

Expose metadata endpoint via configuration option #111

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

Merged

Conversation

c00kiemon5ter
Copy link
Member

Check for configuration option entityid_endpoint.
When set to true the metadata will be served at the URL pointed by the entityid URI.

This commit introduces a new class SAMLBaseModule which serves as a common place for both frontend and backend saml modules.

An example configuration would be:

module: satosa.backends.saml2.SAMLBackend
name: Saml2
config:
  entityid_endpoint: true
  sp_config:
    entityid: <base_url>/<name>/proxy_saml2_backend.xml
    ...

Check for configuration option 'entityid_endpoint'.
When set to true the metadata will be served at the entityid URI.
@johanlundberg
Copy link
Contributor

Even though I like the addition of the SAMLBaseModule (and think other configuration could use that abstraction) I don't know if it is ok to add it just for this config option. Or should we add it and see it as a start for similar config abstractions?

@johanlundberg johanlundberg requested a review from jkakavas July 11, 2017 15:00
@c00kiemon5ter
Copy link
Member Author

The SAMLBaseModule class serves as a common placeholder for common functionality in both the SAML frontend and backend modules. While at the moment it only offers that option, in the future it could hold more information (I plan to do that).

If you do not want to go forward with that, what would the alternative to share code between the frontend and backend modules be? I am open to suggestions, though I definitely do not want to duplicate the code. The only other option I can think of atm, is putting the code in a function in a generic file like utils.py and calling into that (which I feel is uglier).

@jkakavas
Copy link
Member

I feel this is a cleaner way to add the common functionality needed in all SAML SATOSA plugins than any of the alternatives I can think of. And I know that @c00kiemon5ter has more functionality to submit that fits nicely on that base class so I'm all in for accepting this. We can by all means discuss it further though, if anyone has concerns

@johanlundberg
Copy link
Contributor

Just what I hoped you would say, that it's just a start :)

@johanlundberg johanlundberg merged commit b15f5ca into IdentityPython:master Jul 12, 2017
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.

3 participants