[ 
https://issues.apache.org/jira/browse/TS-3222?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14253573#comment-14253573
 ] 

ASF subversion and git services commented on TS-3222:
-----------------------------------------------------

Commit 2710ad047f1ccbd70cc338944f7ab6173097b5eb in trafficserver's branch 
refs/heads/master from [~amc]
[ https://git-wip-us.apache.org/repos/asf?p=trafficserver.git;h=2710ad0 ]

TS-3222: Fix URL print to not put leading zeros in port.


> Port number padded with 0 when Host header has explicit port specification
> --------------------------------------------------------------------------
>
>                 Key: TS-3222
>                 URL: https://issues.apache.org/jira/browse/TS-3222
>             Project: Traffic Server
>          Issue Type: Bug
>            Reporter: Nakagome Tomoyuki
>            Assignee: Alan M. Carroll
>             Fix For: 5.3.0
>
>
> Due to this line in proxy/hdrs/HTTP.cc:1552
> {noformat}
> ui->m_len_port = sprintf(port_buff, "%.5d", hdr->m_port);
> {noformat}
> UrlPrintHack::UrlPrintHack() prepends unnecessary '0's so the port_buff will 
> be 5 chars, like "00080". Suggestion is to remove .5 from the format string 
> and change it to
> {noformat}
> ui->m_len_port = snprintf(port_buff, 6, "%d", hdr->m_port);
> {noformat}
> unless there are some specific reasons for the current code.
> To replicate the symptom, send a request with a Host header having a port 
> number like:
> Host: www.apache.org:80



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to