Bug 788117 - Invalid message is displayed when login
Summary: Invalid message is displayed when login
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: CloudForms Cloud Engine
Classification: Retired
Component: aeolus-conductor
Version: 1.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium
Target Milestone: rc
Assignee: Matt Wagner
QA Contact: wes hayutin
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2012-02-07 14:34 UTC by Shveta
Modified: 2014-08-17 22:27 UTC (History)
5 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-30 17:14:31 UTC
Embargoed:


Attachments (Terms of Use)
header_message (216.17 KB, image/png)
2012-02-07 14:34 UTC, Shveta
no flags Details

Description Shveta 2012-02-07 14:34:37 UTC
Created attachment 559971 [details]
header_message

Description of problem:


Version-Release number of selected component (if applicable):


How reproducible:


Steps to Reproduce:
1. Login as admin , message displayed "You must be logged out to access this page"
2.
3.
  
Actual results:


Expected results:


Additional info:

Comment 1 Matt Wagner 2012-02-07 21:39:53 UTC
This was an unintended consequence of the fix for #782242. We were sometimes setting session[:return_to] (the path to take users to after logging in) to "/login", causing us to attempt to redirect users to the login page after they logged in... Hence that error.

Testing a patch for this now.

Comment 2 Matt Wagner 2012-02-07 21:58:21 UTC
I've sent a patch to the list, though it seems to be stuck in a queue somewhere. Will update when it shows up in the archives... Setting this to POST now, though.

Comment 3 Matt Wagner 2012-02-08 16:56:44 UTC
Pushed to staging:

commit 21bbfbf374bd3e616e41741bcd018c8045f4f933
Author: Matt Wagner <matt.wagner>
Date:   Tue Feb 7 16:30:13 2012 -0500

    BZ 788117 - Don't set session[:return_to] in UserSessions#new
    
    This is already set up by require_user, so this was redundant.
    Additionally, there was a bug where if you logged out, you were sent to /log
    which would set return_to to request.request_uri... which was /login.
    
    Resolves https://bugzilla.redhat.com/show_bug.cgi?id=788117

Comment 4 Aziza Karol 2012-02-09 04:46:03 UTC
This logout message not displayed now.

verified :
rpm -qa | grep aeolus
aeolus-configure-2.5.0-12.el6.noarch
aeolus-conductor-doc-0.8.0-24.el6.noarch
rubygem-aeolus-cli-0.3.0-8.el6.noarch
aeolus-conductor-0.8.0-24.el6.noarch
rubygem-aeolus-image-0.3.0-7.el6.noarch
aeolus-all-0.8.0-24.el6.noarch
aeolus-conductor-daemons-0.8.0-24.el6.noarch


Note You need to log in before you can comment on or make changes to this bug.