[HTTPCLIENT-1092] ClientPNames.VIRTUAL_HOST is used as is; if not provided, the port should be derived from the target URL

• Type: Improvement
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.1.1
• Fix Version/s: 4.1.2
• Component/s: HttpClient (classic)
• Assignee: Unassigned
• Reporter: Sebb
• Created: 18/May/11 10:08
• Updated: 23/Jul/11 09:58
• Resolved: 20/May/11 20:53

Description

The parameter ClientPNames.VIRTUAL_HOST allows the default Host header to be overridden.

Currently the code uses the HttpHost entry as provided, and does not automatically add the port suffix.
This means that user code has to provide the port - but only if it's not the default for the protocol.

It would be simpler for the user if the port were automatically added.

If the user does not provide the port, the code should derive it from the target URL.

If the user does provide a port number, then that should be used (as is done currently).
This allows the user to override the port (if that should ever prove necessary).