Actions
Bug #19017
closedUse request id instead of session id of tagged logging
Difficulty:
Triaged:
Bugzilla link:
Pull request:
Description
#15863 introduced tagged logging. In the review discussion, it was suggested to use session id
if available. Although it seemed as a good idea at the beginning, it has several disadvanteges:
- it treats UI and API requests differently
- it makes it impossible to differentiate requests made from the session at the same time (notifications
is one example, but in general, we should not assume the user is performing on request at a time) - the usability of this as correlation id is questionable, especially with asynchronous actions in place
Updated by The Foreman Bot over 7 years ago
- Status changed from Assigned to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/4410 added
Updated by Ivan Necas over 7 years ago
- Related to Feature #15863: Log request IDs (tagged logging) in production.log added
Updated by Ivan Necas over 7 years ago
- Status changed from Ready For Testing to Closed
- Bugzilla link set to 1408420
Updated by Ivan Necas over 7 years ago
- Subject changed from Use request id instead of session id of lagged logging to Use request id instead of session id of tagged logging
- Category set to Logging
Updated by Ohad Levy over 7 years ago
- Translation missing: en.field_release set to 209
Actions