Bug 788117

Summary: Invalid message is displayed when login
Product: [Retired] CloudForms Cloud Engine Reporter: Shveta <ssachdev>
Component: aeolus-conductorAssignee: Matt Wagner <matt.wagner>
Status: CLOSED CURRENTRELEASE QA Contact: wes hayutin <whayutin>
Severity: medium Docs Contact:
Priority: unspecified    
Version: 1.0.0CC: akarol, athomas, deltacloud-maint, matt.wagner, ssachdev
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-08-30 17:14:31 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Attachments:
Description Flags
header_message none

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