-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
OIDC SSO Re: Issue #246 #1955
OIDC SSO Re: Issue #246 #1955
Changes from 8 commits
2e90782
4674464
284d215
d0d476f
d0d261a
05a4a6c
7f97e7f
635a485
47d5320
4d39197
92b4c7e
ce6a114
7beebe3
bba76c8
519feba
5c7161d
0f6987d
e6ec777
92a0171
2def119
f8e1739
4a76119
79bf741
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -5,9 +5,16 @@ CREATE TABLE collections ( | |
); | ||
|
||
CREATE TABLE organizations ( | ||
uuid VARCHAR(40) NOT NULL PRIMARY KEY, | ||
name TEXT NOT NULL, | ||
billing_email TEXT NOT NULL | ||
uuid VARCHAR(40) NOT NULL PRIMARY KEY, | ||
name TEXT NOT NULL, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These should be added in a separate migration, something like this one https://github.com/dani-garcia/vaultwarden/blob/main/migrations/sqlite/2021-05-11-205202_add_hide_email/up.sql There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I would even say this should be in a separate table by it self and have foreign keys on the org id. |
||
billing_email TEXT NOT NULL, | ||
identifier TEXT NOT NULL, | ||
use_sso BOOLEAN NOT NULL, | ||
callback_path TEXT NOT NULL, | ||
signed_out_callback_path TEXT NOT NULL, | ||
authority TEXT NOT NULL, | ||
client_id TEXT NOT NULL, | ||
client_secret TEXT NOT NULL | ||
); | ||
|
||
CREATE TABLE users_collections ( | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This urlencoding library doesn't seem to be used.