Skip to content

Develop#86

Merged
jeffersonrodrigues92 merged 3 commits intomainfrom
develop
Mar 26, 2026
Merged

Develop#86
jeffersonrodrigues92 merged 3 commits intomainfrom
develop

Conversation

@jeffersonrodrigues92
Copy link
Copy Markdown
Contributor

No description provided.

Replace per-request http.Client{} (3 occurrences) with a package-level shared client using custom transport with ForceAttemptHTTP2: false. The bare http.Client{} used Go's default transport which enables HTTP/2 over HTTPS, causing hpack encoder panic under concurrent goroutine access.

X-Lerian-Ref: 0x1
fix: prevent HTTP/2 hpack panic with shared HTTP client
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 26, 2026

Caution

Review failed

The head commit changed during the review from 292e241 to 0475898.


Comment @coderabbitai help to get the list of available commands and usage tips.

## [2.6.0-beta.1](v2.5.0...v2.6.0-beta.1) (2026-03-26)

### Bug Fixes

* prevent HTTP/2 hpack panic with shared HTTP client ([0716b6f](0716b6f))
@lerian-studio-midaz-push-bot
Copy link
Copy Markdown

🎉 This PR is included in version 2.6.0-beta.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Copy Markdown

@gandalf-at-lerian gandalf-at-lerian left a comment

Choose a reason for hiding this comment

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

Merge develop → main. Fix do HTTP/2 hpack panic já revisado no PR #85. LGTM.

@jeffersonrodrigues92 jeffersonrodrigues92 merged commit 891a135 into main Mar 26, 2026
2 checks passed
@lerian-studio-midaz-push-bot
Copy link
Copy Markdown

🎉 This PR is included in version 2.6.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Copy link
Copy Markdown

@gandalf-at-lerian gandalf-at-lerian left a comment

Choose a reason for hiding this comment

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

Re-approved. LGTM.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants