horizon messages not shown on redirect to logout

Bug #1292624 reported by David Lapsley
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Dashboard (Horizon)
Expired
Medium
Unassigned

Bug Description

When a user is redirected to logout via the success_url of a form/action, any messages generated by the form/action workflow are not shown prior to logout, but instead are shown on the next login.

This is not a problem for any current features, but will become a problem for future features that require a redirect to logout.

David Lyle (david-lyle)
Changed in horizon:
status: New → Confirmed
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to horizon (master)

Fix proposed to branch: master
Review: https://review.openstack.org/96560

Changed in horizon:
assignee: nobody → David Lapsley (dlapsley)
status: Confirmed → In Progress
Revision history for this message
Julie Pichon (jpichon) wrote :

I don't think this patch should be merged, the messages are not displayed on the login page by design and due to security concerns, see https://bugs.launchpad.net/horizon/+bug/1165702 . The awkward cookie-based way to display messages on the login page ( https://blueprints.launchpad.net/horizon/+spec/messages-on-login-page ) was devised to only allow specific, known trusted messages to be shown. Specific features should try and use the function to add messages on logout at https://github.com/openstack/horizon/blob/master/horizon/utils/functions.py .

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on horizon (master)

Change abandoned by Dave Lapsley (<email address hidden>) on branch: master
Review: https://review.openstack.org/96560
Reason: Post-approve was decided that it's better not to merge this. Hopefully this will stop the merge.

Akihiro Motoki (amotoki)
Changed in horizon:
assignee: David Lapsley (dlapsley) → nobody
status: In Progress → New
status: New → Confirmed
Revision history for this message
David Lyle (david-lyle) wrote :

don't we already handle this with logout reason?

Changed in horizon:
status: Confirmed → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for OpenStack Dashboard (Horizon) because there has been no activity for 60 days.]

Changed in horizon:
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.