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

Introduce NewLoggingHTTPTransport and deprecate NewTransport #1006

Merged
merged 16 commits into from Jul 26, 2022

Commits on Jul 25, 2022

  1. .gitignore JetBrains IDE project files

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    e110ba6 View commit details
    Browse the repository at this point in the history
  2. Removing fmtcheck target in Makefile, and instead add lint to t…

    …rigger GolangCI-Lint (what we use today)
    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    6601cf7 View commit details
    Browse the repository at this point in the history
  3. Introducing helper/logging NewLoggingHTTPTransport

    This deprecated the `NewTransport` facility.
    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    6a4c7d9 View commit details
    Browse the repository at this point in the history
  4. Set golangci-lint GH Action to always pick the latest version of gola…

    …ngci-lint executable
    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    a1c98bb View commit details
    Browse the repository at this point in the history
  5. PR review implementation

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    9e04373 View commit details
    Browse the repository at this point in the history
  6. Refactor to make use of textproto.ReadMIMEHeader()

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    acd4997 View commit details
    Browse the repository at this point in the history
  7. Added test coverage to confirm request-body is captured, as well as l…

    …og masking and filtering
    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    a24de60 View commit details
    Browse the repository at this point in the history
  8. Further simplified code, to reduce repetition further

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    e3919f7 View commit details
    Browse the repository at this point in the history
  9. Adding CHANGELOG entry

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    67b58fe View commit details
    Browse the repository at this point in the history
  10. PR review

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    69421fb View commit details
    Browse the repository at this point in the history
  11. Copy the full SHA
    5583b6c View commit details
    Browse the repository at this point in the history
  12. Place the req/res body in a dedicated field, and keep the log message…

    … short (i.e. "Sending HTTP Request / Received HTTP Response")
    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    cb6d577 View commit details
    Browse the repository at this point in the history
  13. Bumping dependency to terraform-plugin-log@v0.7.0

    Ivan De Marino committed Jul 25, 2022
    Copy the full SHA
    3803b15 View commit details
    Browse the repository at this point in the history

Commits on Jul 26, 2022

  1. Remove unnecessary exported constant

    Ivan De Marino committed Jul 26, 2022
    Copy the full SHA
    b075aaf View commit details
    Browse the repository at this point in the history
  2. Add a new field to every HTTP transaction: tf_http_trans_id

    This is a UUID, and it's specific to the pair of HTTP Req/Res that 1 HTTP Transaction executes
    Ivan De Marino committed Jul 26, 2022
    Copy the full SHA
    b54d376 View commit details
    Browse the repository at this point in the history
  3. Removing unnecessary response body byte buffer copying

    Ivan De Marino committed Jul 26, 2022
    Copy the full SHA
    bc3e299 View commit details
    Browse the repository at this point in the history