WIP Login api

fixes #5932

Author: mdellweg

CHANGES/5932.feature

@@ -0,0 +1 @@
+Added a login api endpoint to result in an authorization cookie from any other sort of feasible authentication.

pulpcore/app/apps.py

@@ -265,6 +265,7 @@ def ready(self):
 
 def _populate_access_policies(sender, apps, verbosity, **kwargs):
     from pulpcore.app.util import get_view_urlpattern
+    from pulpcore.app.viewsets import LoginViewSet
 
     try:
         AccessPolicy = apps.get_model("core", "AccessPolicy")
@@ -273,7 +274,8 @@ def _populate_access_policies(sender, apps, verbosity, **kwargs):
             print(_("AccessPolicy model does not exist. Skipping initialization."))
         return
 
-    for viewset_batch in sender.named_viewsets.values():
+    extra_viewsets = [LoginViewSet]
+    for viewset_batch in list(sender.named_viewsets.values()) + [extra_viewsets]:
         for viewset in viewset_batch:
             access_policy = getattr(viewset, "DEFAULT_ACCESS_POLICY", None)
             if access_policy is not None:

pulpcore/app/serializers/__init__.py

@@ -113,6 +113,7 @@
     GroupRoleSerializer,
     GroupSerializer,
     GroupUserSerializer,
+    LoginSerializer,
     NestedRoleSerializer,
     RoleSerializer,
     UserRoleSerializer,

pulpcore/app/serializers/user.py

@@ -3,6 +3,7 @@
 from gettext import gettext as _
 
 from django.contrib.auth import get_user_model
+from django.contrib.auth import login as auth_login
 from django.contrib.auth.models import Permission
 from django.contrib.auth.hashers import make_password
 from django.contrib.auth.password_validation import validate_password
@@ -490,3 +491,14 @@ def validate(self, data):
                         )
                     self.group_role_pks.append(qs.get().pk)
         return data
+
+
+class LoginSerializer(serializers.Serializer):
+    pulp_href = IdentityField(view_name="users-detail")
+    prn = PRNField()
+    username = serializers.CharField(read_only=True)
+
+    def create(self, validated_data):
+        user = self.context["request"].user
+        auth_login(self.context["request"], user)
+        return user

pulpcore/app/urls.py

@@ -22,6 +22,7 @@
 )
 from pulpcore.app.viewsets import (
     ListRepositoryVersionViewSet,
+    LoginViewSet,
     OrphansCleanupViewset,
     ReclaimSpaceViewSet,
 )
@@ -152,6 +153,7 @@ class PulpDefaultRouter(routers.DefaultRouter):
     vs_tree.add_decendent(ViewSetNode(viewset))
 
 special_views = [
+    path("login/", LoginViewSet.as_view()),
     path("repair/", RepairView.as_view()),
     path(
         "orphans/cleanup/",

pulpcore/app/viewsets/__init__.py

@@ -76,6 +76,7 @@
     GroupViewSet,
     GroupRoleViewSet,
     GroupUserViewSet,
+    LoginViewSet,
     RoleViewSet,
     UserViewSet,
     UserRoleViewSet,

pulpcore/app/viewsets/user.py

@@ -1,14 +1,15 @@
 from gettext import gettext as _
 
 from django.contrib.auth import get_user_model
+from django.contrib.auth import logout as auth_logout
 from django.contrib.contenttypes.models import ContentType
 from django.core.exceptions import FieldError
 from django.shortcuts import get_object_or_404
 from django_filters.rest_framework import filters
 from django.db.models import Q, Count
 from django.contrib.auth.models import Permission
 
-from rest_framework import mixins, status
+from rest_framework import generics, mixins, status
 from rest_framework.exceptions import PermissionDenied
 from rest_framework.permissions import SAFE_METHODS
 from rest_framework.response import Response
@@ -24,6 +25,7 @@
     GroupSerializer,
     GroupUserSerializer,
     GroupRoleSerializer,
+    LoginSerializer,
     RoleSerializer,
     UserSerializer,
     UserRoleSerializer,
@@ -422,3 +424,35 @@ class GroupRoleViewSet(
     serializer_class = GroupRoleSerializer
     queryset = GroupRole.objects.all()
     ordering = ("-pulp_created",)
+
+
+class LoginViewSet(generics.CreateAPIView):
+    serializer_class = LoginSerializer
+
+    DEFAULT_ACCESS_POLICY = {
+        "statements": [
+            {
+                "action": ["*"],
+                "principal": "authenticated",
+                "effect": "allow",
+            },
+        ],
+        "creation_hooks": [],
+    }
+
+    @staticmethod
+    def urlpattern():
+        return "login"
+
+    @extend_schema(operation_id="login_read")
+    def get(self, request):
+        return Response(self.get_serializer(request.user).data)
+
+    @extend_schema(operation_id="logout")
+    def delete(self, request):
+        auth_logout(request)
+        return Response(status=204)
+
+
+# Annotate without redefining the post method.
+extend_schema(operation_id="login")(LoginViewSet.post)

pulpcore/tests/functional/api/test_login.py

@@ -0,0 +1,77 @@
+import http
+import pytest
+
+pytestmark = [pytest.mark.parallel]
+
+
+def test_login_read_denies_anonymous(pulpcore_bindings, anonymous_user):
+    with anonymous_user:
+        with pytest.raises(pulpcore_bindings.module.ApiException) as exc:
+            pulpcore_bindings.LoginApi.login_read()
+    assert exc.value.status == 401
+
+
+def test_login_read_returns_admin(pulpcore_bindings):
+    result = pulpcore_bindings.LoginApi.login_read()
+    assert result.username == "admin"
+
+
+def test_login_read_returns_username(pulpcore_bindings, gen_user):
+    user = gen_user()
+    with user:
+        result = pulpcore_bindings.LoginApi.login_read()
+    assert result.username == user.username
+
+
+def test_login_denies_anonymous(pulpcore_bindings, anonymous_user):
+    with anonymous_user:
+        with pytest.raises(pulpcore_bindings.module.ApiException) as exc:
+            pulpcore_bindings.LoginApi.login()
+    assert exc.value.status == 401
+
+
+def test_login_returns_username(pulpcore_bindings, gen_user):
+    user = gen_user()
+    with user:
+        pulpcore_bindings.LoginApi.login()
+        response = pulpcore_bindings.LoginApi.login_with_http_info()
+        if isinstance(response, tuple):
+            # old bindings
+            result, status, headers = response
+        else:
+            # new bindings
+            result = response.data
+            status = response.status
+            headers = response.headers
+    assert status == 201
+    assert result.username == user.username
+    cookie_jar = http.cookies.SimpleCookie(headers["set-cookie"])
+    assert cookie_jar["sessionid"] != ""
+    assert cookie_jar["csrftoken"] != ""
+
+
+def test_logout_removes_sessionid(pulpcore_bindings, gen_user):
+    user = gen_user()
+    with user:
+        response = pulpcore_bindings.LoginApi.logout_with_http_info()
+        if isinstance(response, tuple):
+            # old bindings
+            _, status, headers = response
+        else:
+            # new bindings
+            status = response.status
+            headers = response.headers
+    assert status == 204
+    if "set-cookie" in headers:
+        # TODO It is indeed a bit unclear how to use a session cookie with the bindings.
+        # Revisit after updating the bindings container may be worth a shot.
+        cookie_jar = http.cookies.SimpleCookie(headers["set-cookie"])
+        assert cookie_jar["sessionid"] == ""
+        assert cookie_jar["csrftoken"] != ""
+
+
+def test_logout_denies_anonymous(pulpcore_bindings, anonymous_user):
+    with anonymous_user:
+        with pytest.raises(pulpcore_bindings.module.ApiException) as exc:
+            pulpcore_bindings.LoginApi.logout()
+    assert exc.value.status == 401