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

Prepare authorization interfaces for extraction #2385

Merged
merged 2 commits into from
Mar 30, 2020

Conversation

pgermishuys
Copy link
Contributor

@pgermishuys pgermishuys commented Mar 24, 2020

Changed: Prepared the authorization interfaces for plugin extraction
Added: The following options have been added AuthorizationType and AuthorizationConfig to mirror that of the existing AuthenticationType and AuthenticationConfig.

Partially fixes: https://github.com/EventStore/EventStore.CommercialHA/issues/128

@pgermishuys pgermishuys added the subsystem/core database Issues relating to the core database label Mar 24, 2020
@pgermishuys pgermishuys force-pushed the extract-authentication-provider-plugins branch from 6fb90a2 to f3b8bfe Compare March 24, 2020 08:58
thefringeninja
thefringeninja previously approved these changes Mar 24, 2020
@hayley-jean hayley-jean force-pushed the extract-authentication-provider-plugins branch from 10b730d to 5f1180b Compare March 26, 2020 08:14
thefringeninja
thefringeninja previously approved these changes Mar 26, 2020
jageall
jageall previously approved these changes Mar 26, 2020
@hayley-jean hayley-jean dismissed stale reviews from jageall and thefringeninja via ef918b4 March 27, 2020 14:00
@hayley-jean hayley-jean force-pushed the extract-authentication-provider-plugins branch from ef918b4 to f7bd6b0 Compare March 27, 2020 15:47
@hayley-jean hayley-jean force-pushed the extract-authentication-provider-plugins branch from f7bd6b0 to eb657f6 Compare March 30, 2020 08:11
@jageall jageall merged commit 46c89cd into master Mar 30, 2020
@pgermishuys pgermishuys deleted the extract-authentication-provider-plugins branch April 15, 2020 08:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
subsystem/core database Issues relating to the core database
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants