Since there are multiple actions on the collection, would it be cleaner to use:

collection do
  get :index, :action => :list_products
  put :index, :action => :update_products
end

(Don't have to make this change, I just generally prefer this notation when there are multiple actions that are member/collection...etc)