Add http_debug_logger option

[murakami-ta]

Purpose

Enable masking sensitive information (e.g. Authentication headers) in HTTP debug logs by allowing users to inject a custom logger.

Faraday's :logger middleware also supports options (e.g. filter headers), but exposing them would tightly couple this gem's interface to Faraday's internals. As a first step, only custom logger injection is added.

Overview

• Add http_debug_logger option to faraday_client that is passed to Faraday's :logger middleware
• Add tests for custom logger, default logger, and no-logger behavior

Checklist

• Code compiles correctly
• Created tests which fail without the change (if possible)
• All tests passing
• Extended the README / documentation, if necessary

[Copilot]

Pull request overview

This pull request adds an optional http_debug_logger parameter to the Faraday client builder so callers can supply their own Ruby Logger instance for Faraday’s HTTP debug logging (enabling custom filtering/masking behavior without exposing Faraday-specific logger options in the gem’s public API).

Changes:

• Pass an optional custom logger to Faraday’s :logger response middleware when http_debug: true.
• Add RSpec coverage for custom logger, default (STDOUT) logger, and disabled-logging behavior.
• Document the new http_debug_logger option in the README.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File	Description
lib/trino/client/faraday_client.rb	Threads http_debug_logger through to Faraday’s :logger middleware when http_debug is enabled.
spec/statement_client_spec.rb	Adds tests verifying custom logger output, default STDOUT logging, and no logging when http_debug is false.
README.md	Documents the new http_debug_logger option and its dependency on http_debug: true.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[yuokada]

@murakami-ta Thanks for your contribution. Could you fix the test failures?

[murakami-ta]

@yuokada Thanks for the review! I fixed the test failures. 0976be7

[Copilot]

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.

[yuokada]

LGTM

[yuokada]

@murakami-ta I've merged. We've released a new version below:
https://rubygems.org/gems/trino-client/versions/2.2.5