8000 fix: Django 5 only allows POST requests to log out (#7728) · django-cms/django-cms@c3eb30c · GitHub
[go: up one dir, main page]

Skip to content

Commit c3eb30c

Browse files
authored
fix: Django 5 only allows POST requests to log out (#7728)
* Django 5 only allows POST requests to log out * Update test
1 parent c0f36f9 commit c3eb30c

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

cms/cms_toolbars.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@
2323
from cms.toolbar_base import CMSToolbar
2424
from cms.toolbar_pool import toolbar_pool
2525
from cms.utils import get_language_from_request, page_permissions
26+
from cms.utils.compat import DJANGO_4_2
2627
from cms.utils.conf import get_cms_setting
2728
from cms.utils.i18n import get_language_dict, get_language_tuple
2829
from cms.utils.page_permissions import (
@@ -343,7 +344,7 @@ def add_logout_button(self, parent):
343344
action=admin_reverse('logout'),
344345
active=True,
345346
on_success=on_success,
346-
method='GET',
347+
method='POST',
347348
)
348349

349350
def add_language_menu(self):

cms/tests/test_toolbar.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -630,7 +630,7 @@ def test_hide_toolbar_login_anonymous_setting(self):
630630
def test_admin_logout_staff(self):
631631
with override_settings(CMS_PERMISSION=True):
632632
with self.login_user_context(self.get_staff()):
633-
response = self.client.get('/en/admin/logout/')
633+
response = self.client.post('/en/admin/logout/')
634634
self.assertTrue(response.status_code, 200)
635635

636636
def test_show_toolbar_without_edit(self):

0 commit comments

Comments
 (0)
0