fixes: v9.0.0 fails template creation with ES v7 with ElasticsearchClientException #518 (#533)

Co-authored-by: AlexGoris-KasparSolutions

Author: nenadvicentic

src/Serilog.Sinks.Elasticsearch/Sinks/ElasticSearch/ElasticsearchSinkState.cs

@@ -52,8 +52,6 @@ public static ElasticsearchSinkState Create(ElasticsearchSinkOptions options)
 
         private readonly ElasticsearchVersionManager _versionManager;
 
-        private bool IncludeTypeName => _versionManager.EffectiveVersion.Major >= 7;
-
         public ElasticsearchSinkOptions Options => _options;
         public IElasticLowLevelClient Client => _client;
         public ITextFormatter Formatter => _formatter;
@@ -172,20 +170,16 @@ public void RegisterTemplateIfNeeded()
                     }
                 }
 
-                StringResponse result;                
-                if (_versionManager.EffectiveVersion.Major < 8)
-                { 
-                    result = _client.Indices.PutTemplateForAll<StringResponse>(_templateName, GetTemplatePostData(),
+                var result = _versionManager.EffectiveVersion.Major switch
+                {
+                    < 8 => _client.Indices.PutTemplateForAll<StringResponse>(_templateName, GetTemplatePostData(),
                         new PutIndexTemplateRequestParameters
                         {
-                            IncludeTypeName = IncludeTypeName ? true : (bool?)null
-                        });
-                }
-                else
-                {
+                            IncludeTypeName = _versionManager.EffectiveVersion.Major < 7 ? true : null
+                        }),
                     // Default to version 8 API
-                    result = _client.Indices.PutTemplateV2ForAll<StringResponse>(_templateName, GetTemplatePostData());
-                }
+                    _ => _client.Indices.PutTemplateV2ForAll<StringResponse>(_templateName, GetTemplatePostData()),
+                };
 
                 if (!result.Success)
                 {

test/Serilog.Sinks.Elasticsearch.Tests/Templating/SendsTemplateTests.cs

@@ -42,6 +42,7 @@ public void TemplatePutToCorrectUrl()
         {
             var uri = _templatePut.Item1;
             uri.AbsolutePath.Should().Be("/_template/serilog-events-template");
+            uri.Query.Should().Be(string.Empty);
         }
     }
 }

test/Serilog.Sinks.Elasticsearch.Tests/Templating/Sendsv6TemplateTests.cs

@@ -43,6 +43,7 @@ public void TemplatePutToCorrectUrl()
         {
             var uri = _templatePut.Item1;
             uri.AbsolutePath.Should().Be("/_template/serilog-events-template");
+            uri.Query.Should().Be("?include_type_name=true");
         }
     }
 }

test/Serilog.Sinks.Elasticsearch.Tests/Templating/Sendsv7TemplateTests.cs

@@ -44,6 +44,7 @@ public void TemplatePutToCorrectUrl()
         {
             var uri = _templatePut.Item1;
             uri.AbsolutePath.Should().Be("/_template/serilog-events-template");
+            uri.Query.Should().Be(string.Empty);
         }
     }
 }

test/Serilog.Sinks.Elasticsearch.Tests/Templating/Sendsv8TemplateTests.cs

@@ -43,6 +43,7 @@ public void TemplatePutToCorrectUrl()
         {
             var uri = _templatePut.Item1;
             uri.AbsolutePath.Should().Be("/_index_template/serilog-events-template");
+            uri.Query.Should().Be(string.Empty);
         }
     }
 }