Skip to content

Conversation

fbuetler
Copy link
Contributor

Motivation:

Port of #119

Copy link
Contributor

@tsegismont tsegismont left a comment

Choose a reason for hiding this comment

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

LGTM

@tsegismont
Copy link
Contributor

Can you please look at the test failures? Thanks

@tsegismont
Copy link
Contributor

Can you please fix formatting, it seems a lot lines are not really changes. Thanks

@tsegismont tsegismont added this to the 5.0.0 milestone May 2, 2025
@tsegismont tsegismont changed the title Fix #85 Setting the host header on the request is not reflected on the proxied request May 2, 2025
@fbuetler
Copy link
Contributor Author

fbuetler commented May 2, 2025

Looks like while cherry-picking, some auto-formatting kicked in, unfortunately.

Copy link
Contributor

@tsegismont tsegismont left a comment

Choose a reason for hiding this comment

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

Thanks @fbuetler !

@tsegismont tsegismont merged commit fae97c2 into eclipse-vertx:main May 2, 2025
5 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.

2 participants