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

Tenant credentials should be a separate model #1

Open
fako opened this issue Dec 19, 2018 · 0 comments
Open

Tenant credentials should be a separate model #1

fako opened this issue Dec 19, 2018 · 0 comments

Comments

@fako
Copy link
Contributor

fako commented Dec 19, 2018

Currently the consumer_key and shared_secret get generated on creation of the Tenant. However it would be better if credentials get separated from the Tenant. This has two advantages:

  • Credentials could rotate (feasible with LTI 1.3)
  • Tenants can hold multiple credentials and also multiple types, which allows multiple API key sets per Tenant

To facilitate both of these features we probably need an is_active on the credentials and some kind of type field which indicates whether the credentials are LTI 1, LTI 3 or API X.

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

No branches or pull requests

1 participant