Serilog.Sinks.Elasticsearch don't call any error handlers when logs are rejected

[Evolva]

Does this issue relate to a new feature or an existing bug?

• Bug
• New Feature

What version of Serilog.Sinks.Elasticsearch is affected? Please list the related NuGet package.
8.2.0

Please describe the current behavior?
When an LogEvent is rejected from ElasticSearch none of the configure handlers are called

Please describe the expected behavior?
Configured handlers should be called

Analysis
After some digging it seems that:
https://github.com/serilog/serilog-sinks-elasticsearch/blob/1cf2748b7a85fdd95b2c863ed7d5c54a2252a78c/src/Serilog.Sinks.Elasticsearch/Sinks/ElasticSearch/ElasticSearchSink.cs#L224-L233

don't detect my error because settings is already a Dictionary<string, object>

I rebuild a new version of the sink with

private static bool HasProperty(dynamic settings, string name)
{
    if (settings is IDictionary<string, object>)
        return ((IDictionary<string, object>)settings).ContainsKey(name);
    
    if (settings is System.Dynamic.DynamicObject)
        return ((System.Dynamic.DynamicObject)settings).GetDynamicMemberNames().Contains(name);

    return settings.GetType().GetProperty(name) != null;
}

and it worked.

[mivano]

Thanks for raising this and providing a PR! It is merged to master, trying to collect some more needed changes and bring a package out.