entrouvert
/
django-mellon
16
0
Fork 0
Code Issues Pull Requests Releases Activity

views: show message when logout is refused (#85904) #13

Merged
bdauvergne merged 1 commits from wip/85904-la-detection-same-origin-pour-in into main 2024-01-29 11:02:30 +01:00
Conversation 0 Commits 1 Files Changed 3 +38 -3
bdauvergne commented 2024-01-19 17:44:46 +01:00
Owner
Copy Link

Ok plutôt que deviser du sexe des anges, un patch qui répond à la demande du ticket.

Ok plutôt que deviser du sexe des anges, un patch qui répond à la demande du ticket.
bdauvergne added 1 commit 2024-01-19 17:44:46 +01:00
gitea/django-mellon/pipeline/head This commit looks good Details
3a94680018 views: show message when logout is refused (#85904)
bdauvergne changed title from views: show message when logout is refused (#85904) to WIP: views: show message when logout is refused (#85904) 2024-01-19 17:45:05 +01:00
bdauvergne force-pushed wip/85904-la-detection-same-origin-pour-in from 3a94680018 to b26bb7d8ab 2024-01-19 18:12:45 +01:00 Compare
bdauvergne changed title from WIP: views: show message when logout is refused (#85904) to views: show message when logout is refused (#85904) 2024-01-19 18:36:57 +01:00
ecazenave reviewed 2024-01-22 10:20:25 +01:00
mellon/views.py Outdated
@ -785,6 +785,7 @@ class LogoutView(ProfileMixin, LogMixin, View):
return HttpResponseRedirect(next_url)
else:
self.log.warning('logout refused referer %r is not of the same origin', referer)
messages.warning(request, _('Logout refused referer "%r" is not of the same origin') % referer)
ecazenave commented 2024-01-22 10:20:25 +01:00
Owner
Copy Link

Plutôt message.error puisque c'est est une d'erreur, que la déconnexion ne se fait pas (si j'ai bien compris) ?

Pour la lisibilité du message, je mettrais une petit virgule entre 'refused' et 'referer'.

Plutôt message.error puisque c'est est une d'erreur, que la déconnexion ne se fait pas (si j'ai bien compris) ? Pour la lisibilité du message, je mettrais une petit virgule entre 'refused' et 'referer'.
bdauvergne commented 2024-01-22 10:35:35 +01:00
Author
Owner
Copy Link

Fait. Au passage je suis passé à un template .format() pour que la traduction puisse choisir de ne pas présenter le referer (au final l'usager s'en fout un peu).

Fait. Au passage je suis passé à un template .format() pour que la traduction puisse choisir de ne pas présenter le referer (au final l'usager s'en fout un peu).
bdauvergne force-pushed wip/85904-la-detection-same-origin-pour-in from b26bb7d8ab to 7665e7620c 2024-01-22 10:34:57 +01:00 Compare
bdauvergne requested review from ecazenave 2024-01-22 10:35:41 +01:00
bdauvergne force-pushed wip/85904-la-detection-same-origin-pour-in from 7665e7620c to 51ee9d8cac 2024-01-22 10:41:35 +01:00 Compare
ecazenave approved these changes 2024-01-22 11:10:49 +01:00
bdauvergne merged commit 51ee9d8cac into main 2024-01-29 11:02:30 +01:00
bdauvergne deleted branch wip/85904-la-detection-same-origin-pour-in 2024-01-29 11:02:30 +01:00
Sign in to join this conversation.
Reviewers
No reviewers
ecazenave
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/django-mellon#13
No description provided.
Delete Branch "wip/85904-la-detection-same-origin-pour-in"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?