Project

General

Profile

Bug #3915

Hammer does not handle 404s well

Added by Daniel Lobato Garcia over 6 years ago. Updated almost 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Tomáš Strachota
Category:
Foreman commands (obsolete)
Target version:
1.9.2
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Team Backlog:
Fixed in Releases:
Found in Releases:
In Kanboard:

Description

Hammer should return more informative messages than

Could not create the subnet:
  404 Resource Not Found

or

RestClient::ResourceNotFound (404 Resource Not Found)

when --verbose is set.

https://github.com/theforeman/foreman/pull/1098 needs to be merged for this to happen though.

Related issues

Has duplicate Hammer CLI - Bug #3514: Improve 404 response when a resource isn't foundDuplicate2013-10-25
Blocked by Foreman - Bug #3515: Associated resources not found by ID causes 500 Internal Server ErrorClosed2013-10-25

Associated revisions

Revision e34ae228 (diff)
Added by Tomas Strachota over 6 years ago

fixes #3915 - better handling of 404 exceptions

Revision b3598466
Added by mbacovsky over 6 years ago

Merge pull request #68 from tstrachota/3915_not_found

fixes #3915 - better handling of 404 exceptions

History

#1 Updated by Daniel Lobato Garcia over 6 years ago

  • Related to Bug #3515: Associated resources not found by ID causes 500 Internal Server Error added

#2 Updated by Daniel Lobato Garcia over 6 years ago

  • Related to deleted (Bug #3515: Associated resources not found by ID causes 500 Internal Server Error)

#3 Updated by Daniel Lobato Garcia over 6 years ago

  • Blocked by Bug #3515: Associated resources not found by ID causes 500 Internal Server Error added

#4 Updated by Tomáš Strachota over 6 years ago

  • Has duplicate Bug #3514: Improve 404 response when a resource isn't found added

#5 Updated by Tomáš Strachota over 6 years ago

  • Category changed from Hammer core to Foreman commands (obsolete)
  • Status changed from New to Assigned
  • Assignee set to Tomáš Strachota

Switched the category to Foreman commands as the handling will be foreman specific.

#6 Updated by Tomáš Strachota over 6 years ago

  • Status changed from Assigned to Ready For Testing

#7 Updated by Dominic Cleal over 6 years ago

  • Target version set to 1.9.2

#8 Updated by Anonymous over 6 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

Applied in changeset foreman:hammer-cli-foreman|commit:e34ae228a4c29d01952bf3372ca8598543213b38.

Also available in: Atom PDF