Feature #15874

Log X-Request-ID by default

Added by Dominic Cleal over 1 year ago. Updated about 1 year ago.

Status:Closed
Priority:Normal
Assigned To:Dmitri Dolguikh
Category:Core
Target version:Foreman - Team Ivan Iteration 6
Difficulty: Bugzilla link:
Found in release: Pull request:https://github.com/theforeman/smart-proxy/pull/475
Story points-
Velocity based estimate-
Release1.14.0Release relationshipAuto

Description

The default log configuration of the smart proxy should log the contents of the X-Request-ID header if it's set, which will help debugging as Foreman (or other users) can set it when making requests. This will make it easier to search logs for relevant entries.

Foreman does something similar (but with the session ID) since #15863.


Related issues

Related to Foreman - Feature #10982: Add logging to smart proxy requests (ProxyAPI) Closed 07/02/2015
Related to Smart Proxy - Bug #18115: Smart proxy - 1.14 - Windows - uninitialized constant error Closed 01/17/2017
Duplicated by Smart Proxy - Feature #15877: Print session/request id in foreman-proxy logs Duplicate 07/28/2016

Associated revisions

Revision b39011eb
Added by Dmitri Dolguikh about 1 year ago

Fixes #15874 - added logging of 'X_REQUEST_ID' http headers

History

#1 Updated by Dominic Cleal over 1 year ago

  • Related to Feature #10982: Add logging to smart proxy requests (ProxyAPI) added

#2 Updated by Dominic Cleal over 1 year ago

  • Duplicated by Feature #15877: Print session/request id in foreman-proxy logs added

#3 Updated by Lukas Zapletal over 1 year ago

Make sure this gets recorded in structured format in logger buffer as well, so we can easily pull out logs (e.g. errors) via the API and show them.

#4 Updated by Dmitri Dolguikh about 1 year ago

  • Status changed from New to Assigned
  • Assigned To set to Dmitri Dolguikh

#5 Updated by Dmitri Dolguikh about 1 year ago

  • Target version set to Team Ivan Iteration 6

#6 Updated by The Foreman Bot about 1 year ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/smart-proxy/pull/475 added

#7 Updated by Anonymous about 1 year ago

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

#8 Updated by Dominic Cleal about 1 year ago

  • Release set to 1.14.0

#9 Updated by Dominic Cleal 10 months ago

  • Related to Bug #18115: Smart proxy - 1.14 - Windows - uninitialized constant error added

Also available in: Atom PDF