Skip to content

Conversation

@jfclere
Copy link
Member

@jfclere jfclere commented Sep 21, 2022

Something like:
<Proxy balancer://testAP>
BalancerMember http://localhost:8080/
BalancerMember http://localhost:8081/

is not supported but no error/warning were used, the fix adds a error message and refuses to start httpd.

@jfclere jfclere requested a review from aogburn September 21, 2022 17:13
@rhusar
Copy link
Member

rhusar commented Oct 18, 2022

@aogburn Can you please review? Thanks!

@PaulLodge PaulLodge self-requested a review October 20, 2022 12:43
Copy link
Collaborator

@PaulLodge PaulLodge left a comment

Choose a reason for hiding this comment

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

The PR has been tested in a downstream TS and fails for static workers as expected

@jfclere jfclere merged commit 8d4a322 into modcluster:1.3.x Oct 20, 2022
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.

3 participants