Project

General

Profile

Bug #23785

Do not log large response bodies

Added by Thomas McKay almost 4 years ago. Updated almost 4 years ago.

Status:
Duplicate
Priority:
Normal
Assignee:
Category:
Container
Target version:
Difficulty:
Triaged:
Yes
Bugzilla link:
Pull request:
Fixed in Releases:
Found in Releases:
Red Hat JIRA:

Description

The concerns/api/api_controller.rb has a debug statement in process_action method. When docker pull directly against server (not crane :5000) the blobs (large binary data) is logged. Needs to be a way to either override this method or skip large bodies.


Related issues

Related to Katello - Bug #24370: PUT requests content is logged in info levelClosed

History

#1 Updated by John Mitsch almost 4 years ago

  • Legacy Backlogs Release (now unused) set to 349
  • Assignee set to Thomas McKay
  • Category set to Container

#2 Updated by Stephen Benjamin almost 4 years ago

Also happens for uploading packages/puppet modules, really ugly:

16:41:24 rails.1   | 2018-06-14T16:41:24 [D|app|e1d01] Setting current user thread-local variable to admin
16:41:24 rails.1   | 2018-06-14T16:41:24 [I|kat|e1d01] PUT: https://centos7-devel.zpm.example.com/pulp/a露
C�۾/�R]��j����S���L\��u<��5��;Or���h��ղ��v'��VWWWU�e���e:uܤ>�R��jl5��ݧ��6���k�m��j��~����f���t�_{��D�DZ�Y$߬͜�

#3 Updated by Thomas McKay almost 4 years ago

  • Status changed from New to Duplicate

#4 Updated by Hart Mel almost 4 years ago

  • Related to Bug #24370: PUT requests content is logged in info level added

#5 Updated by Stephen Benjamin almost 4 years ago

  • Status changed from Duplicate to New

Not a duplicate. The fix in #23835 only fixes one case. Uploading RPM's, files, etc to a repository still get logged.

#6 Updated by Stephen Benjamin almost 4 years ago

  • Status changed from New to Duplicate

Er nevermind, just leave this is a duplicate and #24370 can be used for repository uploading.

#7 Updated by Samir Jha almost 4 years ago

  • Target version changed from Katello 3.8.0 to Katello Recycle Bin

Also available in: Atom PDF