From 4f15b24741ca97b6385c7872fe55db0d5d82bf21 Mon Sep 17 00:00:00 2001 From: Steve Gordon Date: Wed, 21 Apr 2021 12:13:40 +0100 Subject: [PATCH] Fix client exception msg to avoid double period --- src/Elasticsearch.Net/Transport/Pipeline/RequestPipeline.cs | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/Elasticsearch.Net/Transport/Pipeline/RequestPipeline.cs b/src/Elasticsearch.Net/Transport/Pipeline/RequestPipeline.cs index 1d4cda51de1..6963056b8ae 100644 --- a/src/Elasticsearch.Net/Transport/Pipeline/RequestPipeline.cs +++ b/src/Elasticsearch.Net/Transport/Pipeline/RequestPipeline.cs @@ -214,8 +214,7 @@ public ElasticsearchClientException CreateClientException( var resource = callDetails == null ? "unknown resource" : $"Status code {statusCode} from: {callDetails.HttpMethod} {callDetails.Uri.PathAndQuery}"; - - + var exceptionMessage = innerException?.Message ?? "Request failed to execute"; var pipelineFailure = data.OnFailurePipelineFailure; @@ -242,8 +241,9 @@ public ElasticsearchClientException CreateClientException( exceptionMessage += ", failed over to all the known alive nodes before failing"; } } + + exceptionMessage += !exceptionMessage.EndsWith(".", StringComparison.Ordinal) ? $". Call: {resource}" : $" Call: {resource}"; - exceptionMessage += $". Call: {resource}"; if (response != null && response.TryGetServerErrorReason(out var reason)) exceptionMessage += $". ServerError: {reason}";