[trafficcontrol] branch master updated: Improve TR logging when a certificate from TO fails to decode

[ne...@apache.org]

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

neuman pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficcontrol.git


The following commit(s) were added to refs/heads/master by this push:
     new 1ed158a  Improve TR logging when a certificate from TO fails to decode
1ed158a is described below

commit 1ed158a698201b3de78ede3c478d1fbc92a1480e
Author: Rawlin Peters <ra...@comcast.com>
AuthorDate: Thu Jun 21 09:05:39 2018 -0600

    Improve TR logging when a certificate from TO fails to decode
    
    This adds the name of the delivery service as well as the hostname of
    the certificate it failed to decode.
---
 .../traffic_router/secure/CertificateDataConverter.java               | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/traffic_router/connector/src/main/java/com/comcast/cdn/traffic_control/traffic_router/secure/CertificateDataConverter.java b/traffic_router/connector/src/main/java/com/comcast/cdn/traffic_control/traffic_router/secure/CertificateDataConverter.java
index 6139c82..bc37f64 100644
--- a/traffic_router/connector/src/main/java/com/comcast/cdn/traffic_control/traffic_router/secure/CertificateDataConverter.java
+++ b/traffic_router/connector/src/main/java/com/comcast/cdn/traffic_control/traffic_router/secure/CertificateDataConverter.java
@@ -41,7 +41,9 @@ public class CertificateDataConverter {
 				x509Chain.toArray(new X509Certificate[x509Chain.size()]), privateKey);
 
 		} catch (Exception e) {
-			log.error("Failed to convert certificate data from traffic ops to handshake data! " + e.getClass().getSimpleName() + ": " + e.getMessage(), e);
+			log.error("Failed to convert certificate data (delivery service = " + certificateData.getDeliveryservice()
+					+ ", hostname = " + certificateData.getHostname() + ") from traffic ops to handshake data! "
+					+ e.getClass().getSimpleName() + ": " + e.getMessage(), e);
 		}
 		return null;
 	}