Merge pull request #1441 from ae-utbm/fix-barman-login

fix: session not properly updated on counter login

counter/tests/test_counter.py

@@ -886,6 +886,28 @@ class TestBarmanConnection(TestCase):
         )
         assert response.status_code == 403
 
+    def test_logout_then_login(self):
+        """Test that the login is successful if it is after a previous logout.
+
+        This is a regression test for #1440
+        """
+        self.client.post(  # login
+            self.login_url, {"username": self.barman.username, "password": "plop"}
+        )
+        self.client.post(  # logout
+            reverse("counter:logout", kwargs={"counter_id": self.counter.id}),
+            data={"user_id": self.barman.id},
+        )
+        response = self.client.post(  # and re-login
+            self.login_url, {"username": self.barman.username, "password": "plop"}
+        )
+        assert response.status_code == 200
+        assert response.headers["HX-Redirect"] == self.detail_url
+        response = self.client.get(
+            self.detail_url, {"username": self.barman.username, "password": "plop"}
+        )
+        assert self.barman in response.wsgi_request.barmen
+
 
 @pytest.mark.django_db
 def test_barman_timeout(client: Client):

counter/views/home.py

@@ -61,6 +61,7 @@ class CounterLoginFragment(FragmentMixin, SingleObjectMixin, FormView):
         user = form.get_user()
         perm = self.object.permanencies.create(user=user, start=timezone.now())
         self.request.session.setdefault(SESSION_PERMANENCES_KEY, []).append(perm.id)
+        self.request.session.modified = True
         self.success_url = reverse(
             "counter:details", kwargs={"counter_id": self.object.id}
         )

pyproject.toml

@@ -64,7 +64,7 @@ prod = [
 ]
 dev = [
     "django-debug-toolbar>=7.0.0,<8",
-    "ipython>=9.15.0,<10.0.0",
+    "ipython>=9.14.1,<10.0.0",
     "pre-commit>=4.6.0,<5.0.0",
     "ruff>=0.15.19,<1.0.0",
     "djhtml>=3.0.11,<4.0.0",