Project

General

Profile

Feature #10982

Add logging to smart proxy requests (ProxyAPI)

Added by Dominic Cleal about 3 years ago. Updated 2 months ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Logging
Target version:
Team Backlog:
Fixed in Releases:
Found in Releases:

Description

We should add a new logger for debugging smart proxy traffic and requests going via ProxyAPI.

Perhaps inside ProxyAPI classes themselves, or something like RestClient.log (so long as it doesn't duplicate compute resource logging, e.g. with oVirt).


Related issues

Related to Smart Proxy - Feature #15874: Log X-Request-ID by defaultClosed2016-07-28
Related to Foreman - Feature #15863: Log request IDs (tagged logging) in production.logClosed2016-07-27
Related to Plugins - Bug #20457: foreman_memcache is not compatible with 1.16Closed2017-07-31
Has duplicate Foreman - Feature #15876: Send request and session ids to foreman-proxy as HTTP headersDuplicate2016-07-28

Associated revisions

Revision 972adab0 (diff)
Added by Dominic Cleal about 1 year ago

fixes #10982 - configure RestClient log to 'proxy' logger

Request/response information is sent to the proxy logger, showing the
URLs called, headers, request body etc.

The X-Request-ID header is now also set in the proxy requests to a
random ID stored in the session, to prevent the user's real session ID
being hijacked from the proxy request data. The ID should be correlated
against the proxy logger.

History

#1 Updated by Dominic Cleal about 2 years ago

  • Status changed from New to Assigned
  • Assignee set to Dominic Cleal

#2 Updated by Dominic Cleal about 2 years ago

#3 Updated by The Foreman Bot about 2 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3683 added

#4 Updated by Dominic Cleal about 2 years ago

  • Related to Feature #15863: Log request IDs (tagged logging) in production.log added

#5 Updated by Dominic Cleal about 2 years ago

  • Has duplicate Feature #15876: Send request and session ids to foreman-proxy as HTTP headers added

#6 Updated by The Foreman Bot about 1 year ago

  • Pull request https://github.com/theforeman/foreman/pull/4702 added

#7 Updated by Daniel Lobato Garcia about 1 year ago

  • Legacy Backlogs Release (now unused) set to 240

#8 Updated by Dominic Cleal about 1 year ago

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

#9 Updated by Ohad Levy about 1 year ago

  • Related to Bug #20457: foreman_memcache is not compatible with 1.16 added

Also available in: Atom PDF