cloudflare / cloudflare-ingress-controller

A Kubernetes ingress controller for Cloudflare's Argo Tunnels
Apache License 2.0
364 stars 55 forks source link

issue-145: allow protocol logging to be enabled #147

Closed mattalberts closed 5 years ago

mattalberts commented 5 years ago

Adds flag proto-log-enable. When enabled, the protocol logger will be given the same log-level as the controller logger.

resolves #145

codecov-io commented 5 years ago

Codecov Report

Merging #147 into master will decrease coverage by 0.49%. The diff coverage is 37.5%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master    #147     +/-   ##
========================================
- Coverage    59.2%   58.7%   -0.5%     
========================================
  Files          16      17      +1     
  Lines        1103    1126     +23     
========================================
+ Hits          653     661      +8     
- Misses        428     443     +15     
  Partials       22      22
Impacted Files Coverage Δ
cmd/argot/main.go 5.63% <0%> (-0.21%) :arrow_down:
internal/argotunnel/tunnel.go 51.39% <100%> (ø) :arrow_up:
internal/argotunnel/protologger.go 44.44% <44.44%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update d260b67...f0b5ef5. Read the comment docs.