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,
                 )
             )
 

Reply via email to