Skip to content

Add per database permissions for the SQL Lab.#885

Merged
bkyryliuk merged 3 commits into
apache:masterfrom
bkyryliuk:db_permissions
Aug 10, 2016
Merged

Add per database permissions for the SQL Lab.#885
bkyryliuk merged 3 commits into
apache:masterfrom
bkyryliuk:db_permissions

Conversation

@bkyryliuk
Copy link
Copy Markdown
Member

This PR adds per DB permissions, changes the permissions check in the select *, run_sql and sql_json endpoints.

Issue: #851

@mistercrunch - please take a look.

Comment thread caravel/views.py Outdated
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oops

@coveralls
Copy link
Copy Markdown

coveralls commented Aug 9, 2016

Coverage Status

Coverage decreased (-0.04%) to 81.435% when pulling 3839c03 on bkyryliuk:db_permissions into b0a1f07 on airbnb:master.

@mistercrunch
Copy link
Copy Markdown
Member

LGTM!

@coveralls
Copy link
Copy Markdown

coveralls commented Aug 10, 2016

Coverage Status

Coverage decreased (-0.04%) to 81.435% when pulling c3ca344 on bkyryliuk:db_permissions into b48101c on airbnb:master.

@bkyryliuk bkyryliuk merged commit d6bb8c6 into apache:master Aug 10, 2016
@bkyryliuk bkyryliuk deleted the db_permissions branch August 10, 2016 00:53
@dpgaspar dpgaspar mentioned this pull request Apr 18, 2019
3 tasks
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 17, 2021
Bumps [mathjs](https://github.com/josdejong/mathjs) from 8.0.1 to 8.1.1.
- [Release notes](https://github.com/josdejong/mathjs/releases)
- [Changelog](https://github.com/josdejong/mathjs/blob/develop/HISTORY.md)
- [Commits](josdejong/mathjs@v8.0.1...v8.1.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 24, 2021
Bumps [mathjs](https://github.com/josdejong/mathjs) from 8.0.1 to 8.1.1.
- [Release notes](https://github.com/josdejong/mathjs/releases)
- [Changelog](https://github.com/josdejong/mathjs/blob/develop/HISTORY.md)
- [Commits](josdejong/mathjs@v8.0.1...v8.1.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 25, 2021
Bumps [mathjs](https://github.com/josdejong/mathjs) from 8.0.1 to 8.1.1.
- [Release notes](https://github.com/josdejong/mathjs/releases)
- [Changelog](https://github.com/josdejong/mathjs/blob/develop/HISTORY.md)
- [Commits](josdejong/mathjs@v8.0.1...v8.1.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
zhaoyongjie pushed a commit to zhaoyongjie/incubator-superset that referenced this pull request Nov 26, 2021
Bumps [mathjs](https://github.com/josdejong/mathjs) from 8.0.1 to 8.1.1.
- [Release notes](https://github.com/josdejong/mathjs/releases)
- [Changelog](https://github.com/josdejong/mathjs/blob/develop/HISTORY.md)
- [Commits](josdejong/mathjs@v8.0.1...v8.1.1)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>

Co-authored-by: dependabot-preview[bot] <27856297+dependabot-preview[bot]@users.noreply.github.com>
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.11.0 First shipped in 0.11.0 labels Feb 19, 2024
qfcwell pushed a commit to qfcwell/superset that referenced this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.11.0 First shipped in 0.11.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants