backoffice: orcidlogin from frontend implemented

* ref: cern-sis/issues-inspire/issues/509
inspirehep · Sep 24, 2024 · 2b6263f · 2b6263f
1 parent fd58651
commit 2b6263f
Show file tree

Hide file tree

Showing 7 changed files with 22 additions and 126 deletions.
diff --git a/backoffice/.envs/local/.django b/backoffice/.envs/local/.django
@@ -24,3 +24,6 @@ AIRFLOW_BASE_URL=http://airflow-webserver:8080
 AIRFLOW_TOKEN=YWlyZmxvdzphaXJmbG93
 
 SERVICENOW_URL=https://cerntraining.service-now.com
+
+# Frontend
+FRONTEND_LOGIN_SUCCESS=http://127.0.0.1:5000/success
diff --git a/backoffice/config/settings/base.py b/backoffice/config/settings/base.py
@@ -127,7 +127,7 @@
 # https://docs.djangoproject.com/en/dev/ref/settings/#auth-user-model
 AUTH_USER_MODEL = "users.User"
 # https://docs.djangoproject.com/en/dev/ref/settings/#login-redirect-url
-LOGIN_REDIRECT_URL = "users:redirect"
+LOGIN_REDIRECT_URL = env("FRONTEND_LOGIN_SUCCESS")
 # https://docs.djangoproject.com/en/dev/ref/settings/#login-url
 LOGIN_URL = "account_login"
 
@@ -331,7 +331,7 @@
 # https://django-allauth.readthedocs.io/en/latest/configuration.html
 ACCOUNT_USER_MODEL_USERNAME_FIELD = None
 # https://django-allauth.readthedocs.io/en/latest/configuration.html
-ACCOUNT_EMAIL_VERIFICATION = "mandatory"
+ACCOUNT_EMAIL_VERIFICATION = False
 # https://django-allauth.readthedocs.io/en/latest/configuration.html
 ACCOUNT_ADAPTER = "backoffice.users.adapters.AccountAdapter"
 # https://django-allauth.readthedocs.io/en/latest/forms.html

diff --git a/backoffice/config/urls.py b/backoffice/config/urls.py
@@ -1,3 +1,4 @@
+from allauth.socialaccount.providers.orcid.views import oauth2_login
 from django.conf import settings
 from django.conf.urls.static import static
 from django.contrib import admin
@@ -8,6 +9,8 @@
 from drf_spectacular.views import SpectacularAPIView, SpectacularSwaggerView
 from rest_framework.authtoken.views import obtain_auth_token
 from rest_framework_simplejwt.views import TokenObtainPairView, TokenRefreshView
+from django.views.decorators.csrf import csrf_exempt
+
 
 urlpatterns = [
     path("", TemplateView.as_view(template_name="pages/home.html"), name="home"),
@@ -18,6 +21,7 @@
     path(settings.ADMIN_URL, admin.site.urls),
     # User management
     path("users/", include("backoffice.users.urls", namespace="users")),
+    path("accounts/orcid/login/", csrf_exempt(oauth2_login), name="orcid_login"),
     path("accounts/", include("allauth.urls")),
     path("", include("django_prometheus.urls")),
 ] + static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)

diff --git a/backoffice/poetry.lock b/backoffice/poetry.lock
diff --git a/backoffice/pyproject.toml b/backoffice/pyproject.toml
@@ -124,7 +124,6 @@ uvicorn = {version = "0.23.2", extras = ["standard"]}
 django = "4.2.6"
 django-environ = "0.11.2"
 django-model-utils = "4.3.1"
-django-allauth = "0.57.0"
 django-crispy-forms = "2.1"
 crispy-bootstrap5 = "0.7"
 django-redis = "5.4.0"
@@ -150,6 +149,7 @@ opensearch-py = "2.6.0"
 djangorestframework-simplejwt = "^5.3.1"
 django-json-widget = "^2.0.1"
 sentry-sdk = "1.19.1"
+django-allauth = {version = "65.0.1", extras = ["headless"]}
 
 [tool.poetry.dev-dependencies]
 factory-boy = "3.3.0"

diff --git a/docker-compose.yaml b/docker-compose.yaml
@@ -105,6 +105,8 @@ services:
     ports:
       - "8000:8000"
     command: sh -c 'poetry run python manage.py migrate && poetry run python manage.py opensearch index rebuild --force && poetry run python manage.py opensearch document index --force && poetry run python manage.py runserver 0.0.0.0:8000'
+    stdin_open: true
+    tty: true
 
   postgres-airflow:
     image: postgres:13

diff --git a/ruff.toml b/ruff.toml
@@ -1,5 +1,5 @@
 target-version = "py311"
-ignore = ["PT009"]
+lint.ignore = ["PT009"]
 
 [lint.flake8-tidy-imports]
 ban-relative-imports = "all"