[trafficserver] branch quic-latest updated: Print error reason on debug log only if it is provided

[ma...@apache.org]

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

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new 730a57d  Print error reason on debug log only if it is provided
730a57d is described below

commit 730a57dda5a197427090c950d192d057582a9849
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Thu Sep 13 10:39:54 2018 +0900

    Print error reason on debug log only if it is provided
---
 iocore/net/QUICNetVConnection.cc | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 55a347b..7be9e29 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1734,9 +1734,8 @@ QUICNetVConnection::_switch_to_closing_state(QUICConnectionErrorUPtr error)
   }
   if (error->msg) {
     QUICConDebug("Reason: %.*s", static_cast<int>(strlen(error->msg)), error->msg);
-  } else {
-    QUICConDebug("Reason was not provided");
   }
+
   this->_connection_error = std::move(error);
   this->_schedule_packet_write_ready();
 
@@ -1763,8 +1762,6 @@ QUICNetVConnection::_switch_to_draining_state(QUICConnectionErrorUPtr error)
   }
   if (error->msg) {
     QUICConDebug("Reason: %.*s", static_cast<int>(strlen(error->msg)), error->msg);
-  } else {
-    QUICConDebug("Reason was not provided");
   }
 
   this->remove_from_active_queue();