Bug 715422 - Opening sync status page directly from URL gives TypeError
Summary: Opening sync status page directly from URL gives TypeError
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: WebUI
Version: 6.0.0
Hardware: Unspecified
OS: Unspecified
unspecified
medium vote
Target Milestone: Unspecified
Assignee: Shannon Hughes
QA Contact: Katello QA List
URL:
Whiteboard:
Depends On:
Blocks: katello-blockers
TreeView+ depends on / blocked
 
Reported: 2011-06-22 20:48 UTC by Jeff Weiss
Modified: 2019-09-26 13:17 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2012-08-22 17:47:13 UTC
Target Upstream Version:


Attachments (Terms of Use)

Description Jeff Weiss 2011-06-22 20:48:27 UTC
Description of problem:


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


How reproducible:


Steps to Reproduce:
1. Log out
2. Open url  http://[host]:3000/sync_management/index
3.
  
Actual results:
TypeError in Sync managementController#index

can't convert nil into String
Rails.root: /usr/lib/katello

Application Trace | Framework Trace | Full Trace
lib/resources/pulp.rb:211:in `+'
lib/resources/pulp.rb:211:in `sync_history'
app/models/glue/pulp/repo.rb:148:in `_get_most_recent_sync_status'
app/models/glue/pulp/repo.rb:102:in `sync_status'
app/controllers/sync_management_controller.rb:60:in `status'
app/controllers/application_controller.rb:185:in `require_org'
Request

Parameters:

None
Show session dump

Show env dump

Expected results:
Gracefully handle error - if login required, redirect to login page.

Additional info:

Comment 1 Shannon Hughes 2011-07-12 16:42:41 UTC
good catch and very odd issue. looks like 'status' is a reserved method name for rails. had to change to 'sync_status' and then update routes and javascript polling to use new naming covention. apparently status was getting called for all index requests. 

ea28466..f4ab09b  master -> master

Comment 2 Corey Welton 2011-08-05 15:52:35 UTC
Verified.


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