Skip to content

Remove dependency on httputil for client hijack test#50270

Merged
thaJeztah merged 1 commit intomoby:masterfrom
dmcgowan:client-hijack-test
Jun 27, 2025
Merged

Remove dependency on httputil for client hijack test#50270
thaJeztah merged 1 commit intomoby:masterfrom
dmcgowan:client-hijack-test

Conversation

@dmcgowan
Copy link
Member

No utils needed for this, the utils only added one extra check for parse and one extra flush for hijack

Signed-off-by: Derek McGowan <derek@mcg.dev>
Copy link
Member

@thaJeztah thaJeztah left a comment

Choose a reason for hiding this comment

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

LGTM

@thaJeztah thaJeztah added status/2-code-review kind/refactor PR's that refactor, or clean-up code labels Jun 27, 2025
@thaJeztah thaJeztah merged commit fe80dce into moby:master Jun 27, 2025
257 of 262 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in go modules transition Jun 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/refactor PR's that refactor, or clean-up code status/2-code-review

Projects

Development

Successfully merging this pull request may close these issues.

3 participants