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

Provide a behaviour for identity providers storage #56

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

adanselm
Copy link

The idea is to de-correlate identity providers storage from the rest of the application.
A Samly.IdpDataStore.Store behaviour was created, as well as a default implementation Samly.IdpDataStore.Config that does exactly the same as before (read from configuration).
A new configuration option was added to customize the IdP storage module to be used:

config :samly, Samly.Provider,
      idp_data_store: MyApp.IdpStore

If not provided, the default one will be used.

This is intended as a first step towards dynamic IdP data storage. Next step will be to try another implementation and check if Samly's public interface provides all functions needed.

…ementation that does exactly the same as before (read from configuration)
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.

1 participant