mirror of https://github.com/apache/superset.git
chore: Cleanup table access check naming (#27772)
This commit is contained in:
parent
5beda309ab
commit
27acc0b133
|
@ -63,7 +63,7 @@ from superset.commands.importers.exceptions import (
|
||||||
from superset.commands.importers.v1.utils import get_contents_from_bundle
|
from superset.commands.importers.v1.utils import get_contents_from_bundle
|
||||||
from superset.constants import MODEL_API_RW_METHOD_PERMISSION_MAP, RouteMethod
|
from superset.constants import MODEL_API_RW_METHOD_PERMISSION_MAP, RouteMethod
|
||||||
from superset.daos.database import DatabaseDAO, DatabaseUserOAuth2TokensDAO
|
from superset.daos.database import DatabaseDAO, DatabaseUserOAuth2TokensDAO
|
||||||
from superset.databases.decorators import check_datasource_access
|
from superset.databases.decorators import check_table_access
|
||||||
from superset.databases.filters import DatabaseFilter, DatabaseUploadEnabledFilter
|
from superset.databases.filters import DatabaseFilter, DatabaseUploadEnabledFilter
|
||||||
from superset.databases.schemas import (
|
from superset.databases.schemas import (
|
||||||
database_schemas_query_schema,
|
database_schemas_query_schema,
|
||||||
|
@ -694,7 +694,7 @@ class DatabaseRestApi(BaseSupersetModelRestApi):
|
||||||
|
|
||||||
@expose("/<int:pk>/table/<path:table_name>/<schema_name>/", methods=("GET",))
|
@expose("/<int:pk>/table/<path:table_name>/<schema_name>/", methods=("GET",))
|
||||||
@protect()
|
@protect()
|
||||||
@check_datasource_access
|
@check_table_access
|
||||||
@safe
|
@safe
|
||||||
@statsd_metrics
|
@statsd_metrics
|
||||||
@event_logger.log_this_with_context(
|
@event_logger.log_this_with_context(
|
||||||
|
@ -757,7 +757,7 @@ class DatabaseRestApi(BaseSupersetModelRestApi):
|
||||||
|
|
||||||
@expose("/<int:pk>/table_extra/<path:table_name>/<schema_name>/", methods=("GET",))
|
@expose("/<int:pk>/table_extra/<path:table_name>/<schema_name>/", methods=("GET",))
|
||||||
@protect()
|
@protect()
|
||||||
@check_datasource_access
|
@check_table_access
|
||||||
@safe
|
@safe
|
||||||
@statsd_metrics
|
@statsd_metrics
|
||||||
@event_logger.log_this_with_context(
|
@event_logger.log_this_with_context(
|
||||||
|
@ -820,7 +820,7 @@ class DatabaseRestApi(BaseSupersetModelRestApi):
|
||||||
@expose("/<int:pk>/select_star/<path:table_name>/", methods=("GET",))
|
@expose("/<int:pk>/select_star/<path:table_name>/", methods=("GET",))
|
||||||
@expose("/<int:pk>/select_star/<path:table_name>/<schema_name>/", methods=("GET",))
|
@expose("/<int:pk>/select_star/<path:table_name>/<schema_name>/", methods=("GET",))
|
||||||
@protect()
|
@protect()
|
||||||
@check_datasource_access
|
@check_table_access
|
||||||
@safe
|
@safe
|
||||||
@statsd_metrics
|
@statsd_metrics
|
||||||
@event_logger.log_this_with_context(
|
@event_logger.log_this_with_context(
|
||||||
|
|
|
@ -30,9 +30,9 @@ from superset.views.base_api import BaseSupersetModelRestApi
|
||||||
logger = logging.getLogger(__name__)
|
logger = logging.getLogger(__name__)
|
||||||
|
|
||||||
|
|
||||||
def check_datasource_access(f: Callable[..., Any]) -> Callable[..., Any]:
|
def check_table_access(f: Callable[..., Any]) -> Callable[..., Any]:
|
||||||
"""
|
"""
|
||||||
A Decorator that checks if a user has datasource access
|
A Decorator that checks if a user has access to a table in a database.
|
||||||
"""
|
"""
|
||||||
|
|
||||||
def wraps(
|
def wraps(
|
||||||
|
|
Loading…
Reference in New Issue