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

ACL for lookups #12728

Open
danc opened this issue Jul 1, 2022 · 4 comments · May be fixed by #15645
Open

ACL for lookups #12728

danc opened this issue Jul 1, 2022 · 4 comments · May be fixed by #15645

Comments

@danc
Copy link
Contributor

danc commented Jul 1, 2022

Hi, (please consider this as a Feature Request)
We applied Authorization model on datasources, as described here : User authentication and authorization · Apache Druid

Once done, a user connecting with the proper credentials can only acces the datasources he’s been granted.
Can we achieve the same on lookups ?

We discovered this while connecting Superset to our druid instance : all lookups seem accessible, ,while only the authorized datasources are accessible from Superset logged with the restricted login.

@suneet-s
Copy link
Contributor

suneet-s commented Jul 1, 2022

This sounds like a great feature request! Something similar was done in #11720 to add authorization to system tables. That pattern could be repeated to add support adding authorization on the lookup namespace

@yokeshwaran1
Copy link

Hi @danc, @suneet-s,

Im interested in contributing this by introducing ACL for lookups, enhancing the Authorization model. Shall I proceed with working on this issue?

@AlbericByte
Copy link
Contributor

hi @yokeshwaran1 are u still working on this task, if not, may i continue to work on this. @danc , @suneet-s

@AlbericByte AlbericByte linked a pull request Jan 9, 2024 that will close this issue
10 tasks
@AlbericByte
Copy link
Contributor

AlbericByte commented Jan 9, 2024

@danc @suneet-s and @FrankChen021
Could you help to review this code when you have time

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

Successfully merging a pull request may close this issue.

4 participants