Skip to content

Commit bbe7a34

Browse files
authored
Merge pull request #169 from FEWS-NET/HEA-746/Add-login-required-to-the-dagster-access-to-redirect-to-login-page
Add loginrequired to the dagster view see HEA-746
2 parents c6cb19a + f246bde commit bbe7a34

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)