This is an automated email from the ASF dual-hosted git repository.
elizabeth pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git
The following commit(s) were added to refs/heads/master by this push:
new bb9f326f9b chore: change security error level (#28641)
bb9f326f9b is described below
commit bb9f326f9b90b72ac92b3e7542d5955ac6755b05
Author: Elizabeth Thompson <[email protected]>
AuthorDate: Wed Jun 5 15:43:22 2024 -0700
chore: change security error level (#28641)
---
superset/security/manager.py | 10 +++++-----
1 file changed, 5 insertions(+), 5 deletions(-)
diff --git a/superset/security/manager.py b/superset/security/manager.py
index 722ac363a0..a67870a398 100644
--- a/superset/security/manager.py
+++ b/superset/security/manager.py
@@ -550,7 +550,7 @@ class SupersetSecurityManager( # pylint:
disable=too-many-public-methods
return SupersetError(
error_type=SupersetErrorType.DASHBOARD_SECURITY_ACCESS_ERROR,
message="You don't have access to this dashboard.",
- level=ErrorLevel.ERROR,
+ level=ErrorLevel.WARNING,
)
def get_chart_access_error_object(
@@ -567,7 +567,7 @@ class SupersetSecurityManager( # pylint:
disable=too-many-public-methods
return SupersetError(
error_type=SupersetErrorType.CHART_SECURITY_ACCESS_ERROR,
message="You don't have access to this chart.",
- level=ErrorLevel.ERROR,
+ level=ErrorLevel.WARNING,
)
@staticmethod
@@ -609,7 +609,7 @@ class SupersetSecurityManager( # pylint:
disable=too-many-public-methods
return SupersetError(
error_type=SupersetErrorType.DATASOURCE_SECURITY_ACCESS_ERROR,
message=self.get_datasource_access_error_msg(datasource),
- level=ErrorLevel.ERROR,
+ level=ErrorLevel.WARNING,
extra={
"link": self.get_datasource_access_link(datasource),
"datasource": datasource.name,
@@ -638,7 +638,7 @@ class SupersetSecurityManager( # pylint:
disable=too-many-public-methods
return SupersetError(
error_type=SupersetErrorType.TABLE_SECURITY_ACCESS_ERROR,
message=self.get_table_access_error_msg(tables),
- level=ErrorLevel.ERROR,
+ level=ErrorLevel.WARNING,
extra={
"link": self.get_table_access_link(tables),
"tables": [str(table) for table in tables],
@@ -2257,7 +2257,7 @@ class SupersetSecurityManager( # pylint:
disable=too-many-public-methods
SupersetError(
error_type=SupersetErrorType.DASHBOARD_SECURITY_ACCESS_ERROR,
message=_("Guest user cannot modify chart payload"),
- level=ErrorLevel.ERROR,
+ level=ErrorLevel.WARNING,
)
)