Feature #1966

Improve UI errors when smart proxy returns 400 bad request

Added by Dominic Cleal almost 5 years ago. Updated over 2 years ago.

Status:Closed
Priority:Normal
Assigned To:Lukas Zapletal
Category:Orchestration
Target version:Sprint 22
Difficulty: Bugzilla link:
Found in release: Pull request:
Story points-
Velocity based estimate-
Release1.5.0Release relationshipAuto

Description

The smart proxy can return 400 bad request HTTP responses for a large number of configuration problems, which are all logged in the smart proxy's proxy.log file. The Foreman UI though only reports the 400 bad request error, but doesn't indicate that this originated from a call to a smart proxy, or which proxy it was.

The UI should be improved to indicate the origin of the HTTP error and preferably a hint to look in the proxy's log files. If possible (and if it doesn't expose confidential data) then the proxy could return a more meaningful error in the body of the HTTP error to be displayed.


Related issues

Related to Foreman - Feature #4772: Display better error messages for DHCP errors Feedback 03/21/2014
Related to Foreman - Feature #7916: Foreman WebUI should give clearer error message when fore... New 10/13/2014
Related to Foreman - Feature #11154: Print error message from Smart-Proxy in addition to the e... New 07/20/2015

Associated revisions

Revision c83e29ac
Added by Lukas Zapletal over 3 years ago

fixes #1966 - improved UI errors for proxy

History

#1 Updated by Lukas Zapletal almost 4 years ago

  • Description updated (diff)
  • Status changed from New to Assigned
  • Assigned To set to Lukas Zapletal
  • Target version set to Sprint 17
  • Release set to 1.4.0
  • Difficulty set to easy

Oh, I have a fix already locally, forgot to push it! Lemme just teardown that patch.

#2 Updated by Lukas Zapletal almost 4 years ago

  • Status changed from Assigned to Ready For Testing

#3 Updated by Dominic Cleal almost 4 years ago

  • Target version changed from Sprint 17 to Sprint 18

#4 Updated by Dmitri Dolguikh almost 4 years ago

  • Target version changed from Sprint 18 to Sprint 19

#5 Updated by Dominic Cleal almost 4 years ago

  • Release deleted (1.4.0)

#6 Updated by Dmitri Dolguikh almost 4 years ago

  • Target version changed from Sprint 19 to Sprint 20

#7 Updated by Dmitri Dolguikh over 3 years ago

  • Target version changed from Sprint 20 to Sprint 21

#8 Updated by Dominic Cleal over 3 years ago

  • Related to Feature #4772: Display better error messages for DHCP errors added

#9 Updated by Dmitri Dolguikh over 3 years ago

  • Target version changed from Sprint 21 to Sprint 22

#10 Updated by Dominic Cleal over 3 years ago

  • Release set to 1.5.0
  • Difficulty deleted (easy)

#11 Updated by Lukas Zapletal over 3 years ago

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

#12 Updated by Dominic Cleal about 3 years ago

  • Related to Feature #7916: Foreman WebUI should give clearer error message when foreman-proxy cannot be contacted added

#13 Updated by Dominic Cleal over 2 years ago

  • Related to Feature #11154: Print error message from Smart-Proxy in addition to the error code added

Also available in: Atom PDF