Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[BUG] Elastic.Serilog.Sinks/README.md is outdated #339

Open
milostns opened this issue Oct 16, 2023 · 1 comment
Open

[BUG] Elastic.Serilog.Sinks/README.md is outdated #339

milostns opened this issue Oct 16, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@milostns
Copy link

milostns commented Oct 16, 2023

I was trying out your nuget package (currently latest version 8.6.1) for Serilog with the example provided at: https://www.nuget.org/packages/Elastic.Serilog.Sinks

I wouldn't compile because there is no ConcurrentConsumers property anymore in the BufferOptions class.

So I guess following line should be updated in the docs:

ConcurrentConsumers = 10

Also a small typo how wer are communicating on this line:

| `Transport` | An instance of `Elastic.Transport` that dictates where and how wer are communicating to. Defaults to `http://localhost:9200` |

@milostns milostns added the bug Something isn't working label Oct 16, 2023
@zerox981
Copy link

Property was renamed to ExportMaxConcurrency for anyone else, looking for a solution without digging through the commit history...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants