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.

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

docs for changes to authorization in #11718 and #11720 #11779

Merged
merged 8 commits into from
Oct 8, 2021
Merged

docs for changes to authorization in #11718 and #11720 #11779

merged 8 commits into from
Oct 8, 2021

Conversation

techdocsmith
Copy link
Contributor

@techdocsmith techdocsmith commented Oct 7, 2021 •
edited
Loading

Additional docs for #11718 and #11720

  • Clarifies you need Datasource WRITE permissions for task and supervisor system tables
  • Adds guidance about this permission to the best practices
  • Adds information about the he SYSTEM_TABLES authorization and information about the impact of setting druid.sql.planner.authorizeSystemTablesDirectly to true.

cc: @kfaraz , @vtlim

This PR has:

  • [ x] been self-reviewed.

@techdocsmith techdocsmith changed the title security updates for #11718 and #11720 docs for changes to authorization in #11718 and #11720 Oct 7, 2021
@kfaraz
Copy link
Contributor

kfaraz commented Oct 7, 2021

Minor comment, otherwise LGTM.
This should be merged only after #11718 is merged.

vtlim
vtlim reviewed Oct 7, 2021
View reviewed changes
docs/operations/security-overview.md Outdated Show resolved Hide resolved
docs/operations/security-overview.md Outdated Show resolved Hide resolved
docs/operations/security-user-auth.md Outdated Show resolved Hide resolved
docs/operations/security-user-auth.md Outdated Show resolved Hide resolved
docs/operations/security-user-auth.md Show resolved Hide resolved
@techdocsmith
Update security-overview.md
2baea2b 
add suggestion for environment variable dynamic config provider
@clintropolis clintropolis merged commit 3ecbd3a into apache:master Oct 8, 2021
@abhishekagarwal87 abhishekagarwal87 added this to the 0.23.0 milestone May 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@abhishekagarwal87 abhishekagarwal87 abhishekagarwal87 left review comments

@vtlim vtlim vtlim left review comments

@clintropolis clintropolis clintropolis approved these changes

@kfaraz kfaraz kfaraz approved these changes

Assignees
No one assigned
Labels
Area - Documentation
Projects
None yet
Milestone
0.23.0
Development

Successfully merging this pull request may close these issues.
5 participants
@techdocsmith @kfaraz @abhishekagarwal87 @clintropolis @vtlim