Prevent double port assignment within ApacheProxyStrategy

[ZuSe]

Resolves: #2856

[ZuSe]

How can I retrigger that test? It failed coz of a 502.

[ZuSe]

PUSH

[jkiddo]

@jamesagnew / @tadgh community would appreciate if you could look at this

[jamesagnew]

I'm not sure I understand what's changing - Could a test be written to exercise the fix? Happy to merge with that in place.

[ZuSe]

Hi @jamesagnew it's explained in the original ticket that I linked.

hapifhir/hapi-fhir-jpaserver-starter#258

No idea how to write a test for it as I would need to make the whole test-suite run on a kubernetes cluster with respective load balancers / network service in place, right?

[jamesagnew]

A simple test that uses Mockito to mock the HttpServletRequest would be the best thing - You can use mockito to have the request supply the various possible input values then ensure that the return value is appropriate.