Skip to content

fix(config): Only warn about deprecated hackney options when explicitly configured#985

Merged
solnic merged 1 commit intogetsentry:masterfrom
zeminlu:fix/hackney-warnings
Feb 6, 2026
Merged

fix(config): Only warn about deprecated hackney options when explicitly configured#985
solnic merged 1 commit intogetsentry:masterfrom
zeminlu:fix/hackney-warnings

Conversation

@zeminlu
Copy link
Contributor

@zeminlu zeminlu commented Feb 5, 2026

Summary

  • Remove the deprecated: directive from hackney options in NimbleOptions schema
  • Handle deprecation warnings manually, only when users explicitly configure hackney options
  • Skip warnings when using default values with the Finch client

Test plan

  • All existing tests pass (523 tests, 0 failures)
  • Config tests specifically pass (23 tests)

Fixes #984

🤖 Generated with Claude Code

#skip-changelog

…ly configured

Remove the `deprecated:` directive from hackney options in NimbleOptions schema
and handle deprecation warnings manually. This ensures warnings are only shown
when users explicitly configure hackney options, not when using default values
with the Finch client.

Add tests to verify the warning behavior.

Fixes getsentryGH-984

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@zeminlu zeminlu force-pushed the fix/hackney-warnings branch from bf1e18c to 5d9eddb Compare February 5, 2026 20:09
@solnic solnic self-requested a review February 6, 2026 11:10
Copy link
Collaborator

@solnic solnic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is super helpful, thank you!!!! <3

@solnic solnic merged commit df8e067 into getsentry:master Feb 6, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Hackney deprecation warnings shown even when using default Finch client

2 participants