diff --git a/superset/security/manager.py b/superset/security/manager.py index b4bc0c6103d..d03da750796 100644 --- a/superset/security/manager.py +++ b/superset/security/manager.py @@ -580,7 +580,7 @@ class SupersetSecurityManager( # pylint: disable=too-many-public-methods """ return ( - f"This endpoint requires the datasource {datasource.name}, " + f"This endpoint requires the datasource {datasource.id}, " "database or `all_datasource_access` permission" ) @@ -612,7 +612,7 @@ class SupersetSecurityManager( # pylint: disable=too-many-public-methods level=ErrorLevel.WARNING, extra={ "link": self.get_datasource_access_link(datasource), - "datasource": datasource.name, + "datasource": datasource.id, }, ) diff --git a/tests/integration_tests/datasource/api_tests.py b/tests/integration_tests/datasource/api_tests.py index 6f25a0c81a9..d9f3650793f 100644 --- a/tests/integration_tests/datasource/api_tests.py +++ b/tests/integration_tests/datasource/api_tests.py @@ -129,7 +129,7 @@ class TestDatasourceApi(SupersetTestCase): response = json.loads(rv.data.decode("utf-8")) self.assertEqual( response["message"], - "This endpoint requires the datasource virtual_dataset, " + f"This endpoint requires the datasource {table.id}, " "database or `all_datasource_access` permission", )