Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Improve and simplify logging #4194

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Commits on Nov 19, 2024

  1. chore: Improve and simplify logging

    * Card ID: CCT-705
    
    This patch will help us with identifying what is happening at given
    times.
    
    - Include phase, version and arguments on each phase load.
      Some of the information may not be necessary, as newer builds of
      insights-client contain logs that carry the information about phase
      and version. However, older builds do not, and these bits are valuable
      when debugging.
    
    - Include payloads, headers and file attachments in the log message, if
      present. This should make it easier to debug PATCH calls.
      BASIC authentication credentials are not included in these headers,
      the library adds them later when constructing the request.
    
    - Disable logging of HTTP responses for checkin. It is verbose and do
      not need to be logged in a file every time.
    
    Signed-off-by: mhorky <mhorky@redhat.com>
    m-horky committed Nov 19, 2024
    Configuration menu
    Copy the full SHA
    aefa1e2 View commit details
    Browse the repository at this point in the history