[HTTPCORE-326] HttpHost class should not add port number if port is default

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.2.2, 4.2.3
• Fix Version/s: 4.2.3
• Component/s: HttpCore
• Assignee: Unassigned
• Reporter: Karl Wright
• Created: 09/Jan/13 22:00
• Updated: 09/Jan/13 22:02
• Resolved: 09/Jan/13 22:02

Description

The following method:

    public String toHostString() {
        if (this.port != -1) {
            //the highest port number is 65535, which is length 6 with the addition of the colon
            StringBuilder buffer = new StringBuilder(this.hostname.length() + 6);
            buffer.append(this.hostname);
            buffer.append(":");
            buffer.append(Integer.toString(this.port));
            return buffer.toString();
        } else {
            return this.hostname;
        }
    }

... always appends a port number. Unfortunately, some sites don't accept a port number. See CONNECTORS-604.