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

Sql and json interceptor #28

Open
wants to merge 94 commits into
base: main
Choose a base branch
from
Open

Sql and json interceptor #28

wants to merge 94 commits into from

Conversation

gunjdesai
Copy link

@gunjdesai gunjdesai commented Nov 14, 2021

Add Complii Permissions Interceptor for SQL Queries and Non SQL Queries

SUMMARY
SQL Editor queries once fired will be first validated on complii's end whether the user has permission to access these api's or not

betodealmeida and others added 30 commits February 26, 2021 16:50
* feat: add Firebird DB engine spec

* Add dep to setup.py

* Fix lint

* Add tests

* Remove uneeded code

* Fix old bug
…pache#13350)

* Reset sqlalchemy connection pool on celery process fork

* Fix race condition with async chart loading state

* pylint: ignore

* prettier
…WINDOW option (apache#13157)

* fix: thumbnails and reports will be use WEBDRIVER_WINDOW option

* changes reformatted

* config change reverted. thumbnails sizes changed to original

* typo fix

* bugfix

defining defaults in thumbnails.py caused thumbnail caches invalidated.
they moved to init.

Co-authored-by: Ibrahim Ercan <ibrahim.ercan@vlmedia.com.tr>
* bump pyarrow and pandas

* remove df copy
* fix: url shortner invalid input

* fix lint
* add file for users to see status of feature flags

* move comment

* update flags per feedback

* move to subfolder and improve docs

* move CODE_OF_CONDUCT to root
* change log with section

* add risk flag for showing risk pull request

* update changlog.md

* lint mypy

* small fixes
* docs: added release notes for 1.1

* fixed link

* incorporated villes excellent feedback
* Update docs for 1.1

* Update CONTRIBUTING.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update UPDATING.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update UPDATING.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

* Update UPDATING.md

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>

Co-authored-by: Michael S. Molina <70410625+michael-s-molina@users.noreply.github.com>
…pache#13735)

* fix: Escape csv content during downloads

* Reuse CsvResponse object

* Use correct mimetype for csv responses

* Ensure that headers are also escaped

* Update escaping logic
)

* add 1.1 cherries to change log

* one more cherry
@github-actions
Copy link

github-actions bot commented Nov 14, 2021

Docker tag: sql-and-json-interceptor-ff8aa12dc44391988bce94e9cdb6713c083ee74c

superset/utils/permissions_manager.py Outdated Show resolved Hide resolved
superset/config.py Outdated Show resolved Hide resolved
superset/config.py Outdated Show resolved Hide resolved
@gunjdesai gunjdesai self-assigned this Nov 26, 2021
@gunjdesai gunjdesai added the type/change_request Existing feature or work in progress change request label Nov 26, 2021
@gunjdesai
Copy link
Author

Dont Merge Yet, found an issue

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/change_request Existing feature or work in progress change request
Projects
None yet
Development

Successfully merging this pull request may close these issues.