summaryrefslogtreecommitdiffstats
path: root/rpc
diff options
context:
space:
mode:
authorRaghavendra Bhat <raghavendra@redhat.com>2016-05-06 10:03:53 -0400
committerAtin Mukherjee <amukherj@redhat.com>2016-08-03 22:10:26 -0700
commit2c97cac30a706fc3458ed557a9fe98b8d2d32ea9 (patch)
treee3162eb6db31ed64ebb1f5e14fa9fdad6e1821fa /rpc
parent058ccf9520794280f3fc254de00e3f604e3cfbb7 (diff)
socket: log the client identifier in ssl connect
Backport of commit d308fb5e152d8c908bf4f5da81f553fbe3d0400a > Change-Id: I4b463ecafb66de16cbe7ed23fae800bb1204f829 > BUG: 1333912 > Signed-off-by: Raghavendra Bhat <raghavendra@redhat.com> > Reviewed-on: http://review.gluster.org/14242 > Tested-by: Vijay Bellur <vbellur@redhat.com> > NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> > CentOS-regression: Gluster Build System <jenkins@build.gluster.org> > Reviewed-by: Jeff Darcy <jdarcy@redhat.com> > Smoke: Gluster Build System <jenkins@build.gluster.org> > (cherry picked from commit d308fb5e152d8c908bf4f5da81f553fbe3d0400a) Change-Id: I2a57a206edab3e0c05ce28c299d78264c9a33d8b BUG: 1351933 Signed-off-by: Mohit Agrawal <moagrawa@redhat.com> Reviewed-on: http://review.gluster.org/14844 NetBSD-regression: NetBSD Build System <jenkins@build.gluster.org> CentOS-regression: Gluster Build System <jenkins@build.gluster.org> Smoke: Gluster Build System <jenkins@build.gluster.org> Reviewed-by: Prashanth Pai <ppai@redhat.com> Reviewed-by: Atin Mukherjee <amukherj@redhat.com>
Diffstat (limited to 'rpc')
-rw-r--r--rpc/rpc-transport/socket/src/socket.c7
1 files changed, 6 insertions, 1 deletions
diff --git a/rpc/rpc-transport/socket/src/socket.c b/rpc/rpc-transport/socket/src/socket.c
index ac164760399..8b49d824524 100644
--- a/rpc/rpc-transport/socket/src/socket.c
+++ b/rpc/rpc-transport/socket/src/socket.c
@@ -347,11 +347,16 @@ ssl_setup_connection (rpc_transport_t *this, int server)
NID_commonName, peer_CN, sizeof(peer_CN)-1);
peer_CN[sizeof(peer_CN)-1] = '\0';
gf_log(this->name,GF_LOG_INFO,"peer CN = %s", peer_CN);
+ gf_log (this->name, GF_LOG_INFO,
+ "SSL verification succeeded (client: %s)",
+ this->xl_private?this->xl_private->client_uid:"");
return gf_strdup(peer_CN);
/* Error paths. */
ssl_error:
- gf_log(this->name,GF_LOG_ERROR,"SSL connect error");
+ gf_log (this->name, GF_LOG_ERROR,
+ "SSL connect error (client: %s)",
+ this->xl_private?this->xl_private->client_uid:"");
ssl_dump_error_stack(this->name);
free_ssl:
SSL_free(priv->ssl_ssl);