Skip to content

Commit f246bde

Browse files
committed
Add loginrequired to the dagster view see HEA-746
1 parent 0e8206c commit f246bde

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

apps/common/views.py

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,11 @@
11
import os
22

3-
from django.contrib.auth.mixins import PermissionRequiredMixin
3+
from django.contrib.auth.mixins import LoginRequiredMixin, PermissionRequiredMixin
44
from revproxy.views import ProxyView
55

66

7-
class DagsterProxyView(PermissionRequiredMixin, ProxyView):
7+
class DagsterProxyView(LoginRequiredMixin, PermissionRequiredMixin, ProxyView):
8+
login_url = "/admin/login/"
89
upstream = f"{os.environ.get('DAGSTER_WEBSERVER_URL')}/{os.environ.get('DAGSTER_WEBSERVER_PREFIX')}/"
910
permission_required = "common.access_dagster_ui"
10-
raise_exception = True
11+
raise_exception = False

0 commit comments

Comments
 (0)