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.

Sign up for GitHub

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

Jump to bottom

feat: handle secondary roles in the session #83

Merged

hantmac merged 4 commits into databendlabs:main from flaneur2020:restrict-role

Nov 20, 2023

Member

flaneur2020 commented Nov 20, 2023 •

edited

Loading

this PR made the following changes:

handle secondary roles in the session state, which could be changed by SET SECONDARY ROLES ALL|NONE
when setting role in the config, restricting the other granted roles not take effect by default.

flaneur2020 added 4 commits

November 20, 2023 18:21


          rename SessionConf to SessionState

d2a0721


          save session state

e639d04


          restrict the secondary roles

b5b6683


          add more comments

0df9710

hantmac approved these changes

View reviewed changes

hantmac merged commit 897d3cf into databendlabs:main

1 check passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet