Project

General

Profile

Bug #13599

Proxy exception messages are truncated

Added by Ohad Levy over 4 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Smart Proxy
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

all proxy exceptions that uses the wrapped exception hide important information, and do not even log to the log file the full message.

looking at the code, I can't really see any usefulness to the WrappedException class? maybe it should be removed?

wrapped.png View wrapped.png 14 KB Ohad Levy, 02/08/2016 07:56 AM
Wrapped

Associated revisions

Revision eda35acd (diff)
Added by Shimon Shtein about 3 years ago

Fixes #13599 - Wrapped exception truncate moved to client

Removed message truncation from server side, and moved it to client
side - the message is truncated with ellipsis. On mouse over full
message is presented as popover.

This commit required a small change to `data-ajax-url` handler in client
to enable proper tooltip presentation in HTML loaded by AJAX.

History

#1 Updated by Dominic Cleal over 4 years ago

  • Subject changed from proxy exceptions are not shown to Proxy exception messages are truncated
  • Category set to Smart Proxy

#2 Updated by The Foreman Bot over 4 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Shlomi Zadok
  • Pull request https://github.com/theforeman/foreman/pull/3165 added

#3 Updated by Dominic Cleal about 4 years ago

  • Status changed from Ready For Testing to New
  • Assignee deleted (Shlomi Zadok)
  • Pull request deleted (https://github.com/theforeman/foreman/pull/3165)

Pull request closed due to inactivity.

#4 Updated by The Foreman Bot about 3 years ago

  • Status changed from New to Ready For Testing
  • Assignee set to Shimon Shtein
  • Pull request https://github.com/theforeman/foreman/pull/4804 added

#5 Updated by Shimon Shtein about 3 years ago

  • Bugzilla link set to 1418305

#6 Updated by Ivan Necas about 3 years ago

  • Status changed from Ready For Testing to Closed
  • Legacy Backlogs Release (now unused) set to 296

Also available in: Atom PDF