influxdata / influxdb-client-swift

InfluxDB (v2+) Client Library for Swift
https://influxdata.github.io/influxdb-client-swift/
MIT License
27 stars 8 forks source link

feat: add logging HTTP requests and responses #52

Closed bednar closed 2 years ago

bednar commented 2 years ago

Proposed Changes

  1. Added logging for HTTP requests and responses.
  2. Redacted authorization header from log output

Checklist

codecov-commenter commented 2 years ago

Codecov Report

Merging #52 (26ef3fa) into master (976e116) will increase coverage by 23.05%. The diff coverage is 40.90%.

:exclamation: Current head 26ef3fa differs from pull request most recent head 5be7bf8. Consider uploading reports for the commit 5be7bf8 to get more accurate results

@@             Coverage Diff             @@
##           master      #52       +/-   ##
===========================================
+ Coverage   42.19%   65.25%   +23.05%     
===========================================
  Files         144       17      -127     
  Lines       10354     1321     -9033     
===========================================
- Hits         4369      862     -3507     
+ Misses       5985      459     -5526     
Impacted Files Coverage Δ
Sources/InfluxDBSwift/Internal/LoggingHelper.swift 29.72% <29.72%> (ø)
Sources/InfluxDBSwift/InfluxDBClient.swift 83.33% <100.00%> (+0.57%) :arrow_up:
...wiftApis/Generated/URLSessionImplementations.swift
Tests/InfluxDBSwiftTests/InfluxDBClientTests.swift
Tests/InfluxDBSwiftTests/XCTestManifests.swift
...s/InfluxDBSwiftApis/Generated/APIs/HealthAPI.swift
...SwiftApis/Generated/Models/PostBucketRequest.swift
...wiftApis/Generated/Models/PatchBucketRequest.swift
...rces/InfluxDBSwiftApis/Generated/Models/Logs.swift
...es/InfluxDBSwiftApis/Generated/APIs/UsersAPI.swift
... and 120 more

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 976e116...5be7bf8. Read the comment docs.