Project

General

Profile

Actions

Bug #5638

closed

Make Katello api error rendering consistent with Foreman's

Added by dustin tsang almost 10 years ago. Updated over 4 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
-
Category:
API
Target version:
-
Difficulty:
hard
Triaged:
Yes
Fixed in Releases:
Found in Releases:

Description

Right now a lot the error messages are rendered differently across Foreman and Katello's APIs
Also, the use of the attribute displayMessage should be limited to the candlepin_proxies_controller where we need to return messages to RHSM in the specific format that it requires.

Forman: https://github.com/theforeman/foreman/blob/develop/app/controllers/api/v2/base_controller.rb#L73
Katello: https://github.com/Katello/katello/blob/master/app/lib/katello/api/v2/error_handling.rb (respond_for_exception method).

So that the api errors are presented consistently across Foreman/Katello, It would be ideal for Katello to share rendering code.

Actions #1

Updated by Eric Helms almost 10 years ago

  • Target version set to 45
  • Triaged changed from No to Yes
Actions #2

Updated by Eric Helms almost 10 years ago

  • Target version changed from 45 to 49
Actions #3

Updated by Eric Helms over 9 years ago

  • Target version changed from 49 to 54
Actions #4

Updated by Eric Helms over 9 years ago

  • Bugzilla link set to 1125401
Actions #5

Updated by Eric Helms over 9 years ago

  • Target version deleted (54)
Actions #6

Updated by Eric Helms over 8 years ago

  • translation missing: en.field_release set to 114
Actions #7

Updated by John Mitsch over 4 years ago

  • Status changed from New to Rejected
  • Target version deleted (Katello Backlog)

Thanks for reporting this issue. This issue was created over 4 years ago and hasn't seen an update in 1 year. We are closing this in an effort to keep a realistic backlog. Please open up a new issue that includes a link to this issue if you feel this still needs to be addressed. We can then triage the new issue and reassess.

Actions

Also available in: Atom PDF