This is an automated email from the ASF dual-hosted git repository.

bcall pushed a commit to branch 8.0.x
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/8.0.x by this push:
     new 0f10c1d  Fix TSHttpTxnEffectiveUrlStringGet to correctly set the port 
when it is non-standard.
0f10c1d is described below

commit 0f10c1d95981a72778777bfece0193d64bcad412
Author: Alan M. Carroll <a...@apache.org>
AuthorDate: Mon Jan 28 16:31:10 2019 -0600

    Fix TSHttpTxnEffectiveUrlStringGet to correctly set the port when it is 
non-standard.
    
    (cherry picked from commit b9f29f0c096e95a621f022c6c30694fd949bd811)
---
 proxy/hdrs/HTTP.cc | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/proxy/hdrs/HTTP.cc b/proxy/hdrs/HTTP.cc
index 3858bd0..c30efbe 100644
--- a/proxy/hdrs/HTTP.cc
+++ b/proxy/hdrs/HTTP.cc
@@ -1702,6 +1702,7 @@ class UrlPrintHack
         ink_assert(nullptr == ui->m_ptr_port); // shouldn't be set if not in 
URL.
         ui->m_ptr_port    = m_port_buff;
         ui->m_len_port    = snprintf(m_port_buff, sizeof(m_port_buff), "%d", 
hdr->m_port);
+        ui->m_port        = hdr->m_port;
         m_port_modified_p = true;
       } else {
         m_port_modified_p = false;
@@ -1725,6 +1726,7 @@ class UrlPrintHack
       if (m_port_modified_p) {
         ui->m_len_port = 0;
         ui->m_ptr_port = nullptr;
+        ui->m_port     = 0;
       }
       if (m_host_modified_p) {
         ui->m_len_host = 0;

Reply via email to