elastic / ecs-dotnet

https://www.elastic.co/guide/en/ecs-logging/dotnet/current/setup.html
Apache License 2.0
108 stars 54 forks source link

[BUG] Recently updated dependency Elastic.Transport 0.4.18 break the use of ElasticsearchSinkOptions #363

Closed GuillaumeBau closed 3 months ago

GuillaumeBau commented 4 months ago

ECS integration/library project(s) (e.g. Elastic.CommonSchema.Serilog): Elastic.Serilog.Sinks

ECS schema version (e.g. 1.4.0): 8.6.1

ECS .NET assembly version (e.g. 1.4.2): 8.6.1

Elasticsearch version (if applicable):

.NET framework / OS: .NET 6

Description of the problem, including expected versus actual behavior: The new Elastic.Transport version introduce the ITransport Interface (https://github.com/elastic/elasticsearch-net/pull/8061) Doing "new ElasticsearchSinkOptions(ITransport)" cause the error CS7069 Reference to type 'HttpTransport' claims it is defined in 'Elastic.Transport', but it could not be found

image

Works with Elastic.Clients.Elasticsearch 8.12.0, the error was introduce when updating to 8.12.1

cbusjay commented 4 months ago

I have this same issue.

Mpdreamz commented 4 months ago

Thank you for raising this @GuillaumeBau we're addressing this here first:

https://github.com/elastic/elastic-ingest-dotnet/pull/47

And will shortly after update all references for the ecs-dotnet libraries as well.

Mpdreamz commented 4 months ago

This has been addressed here: https://github.com/elastic/ecs-dotnet/pull/364