Error on calling ElasticSearch.Nodes.InfoAsync from a 7.3 client to a 7.4 Node.

[MartinDemberger]

NEST: 7.3.0

Elasticsearch version: ElasticStack 7.4.2

When I call InfoAsync I get an exception:

Elasticsearch.Net.UnexpectedElasticsearchClientException: Der angeforderte Wert "voting_only" konnte nicht gefunden werden. ---> System.ArgumentException: Der angeforderte Wert "voting_only" konnte nicht gefunden werden.
   bei System.Enum.TryParseEnum(Type enumType, String value, Boolean ignoreCase, EnumResult& parseResult)
   bei System.Enum.Parse(Type enumType, String value, Boolean ignoreCase)
   bei Elasticsearch.Net.Utf8Json.Formatters.EnumFormatter`1.Deserialize(JsonReader& reader, IJsonFormatterResolver formatterResolver)
   bei Elasticsearch.Net.Utf8Json.Formatters.ListFormatter`1.Deserialize(JsonReader& reader, IJsonFormatterResolver formatterResolver)
   bei Deserialize(Object[] , JsonReader& , IJsonFormatterResolver )
   bei Elasticsearch.Net.Utf8Json.Formatters.DictionaryFormatterBase`5.Deserialize(JsonReader& reader, IJsonFormatterResolver formatterResolver)
   bei Deserialize(Object[] , JsonReader& , IJsonFormatterResolver )
   bei Elasticsearch.Net.Utf8Json.JsonSerializer.Deserialize[T](Byte[] bytes, Int32 offset, IJsonFormatterResolver resolver)
   bei Elasticsearch.Net.Utf8Json.JsonSerializer.<DeserializeAsync>d__29`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   bei Elasticsearch.Net.ResponseBuilder.<SetBodyAsync>d__6`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   bei Elasticsearch.Net.ResponseBuilder.<ToResponseAsync>d__3`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   bei Elasticsearch.Net.HttpWebRequestConnection.<RequestAsync>d__5`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   bei Elasticsearch.Net.RequestPipeline.<CallElasticsearchAsync>d__57`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei Elasticsearch.Net.Transport`1.<RequestAsync>d__15`1.MoveNext()
   --- Ende der internen Ausnahmestapelüberwachung ---
   bei Elasticsearch.Net.Transport`1.<RequestAsync>d__15`1.MoveNext()
--- Ende der Stapelüberwachung vom vorhergehenden Ort, an dem die Ausnahme ausgelöst wurde ---
   bei System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   bei System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   bei System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()

Steps to Reproduce:

1. Create a ElasticClient with a connection to a ElasticSearch 7.3.0
2. Call Nodes.InfoAsync()

[russcam]

Thanks for opening, @MartinDemberger. The NodeRole enum is missing the VotingOnly node role. Will add it for the next release.

[russcam]

opened #4276 to address

[MartinDemberger]

Thinks for the fix, @russcam.
This only saves the error if you use the newer NEST-client. Wouldn't it be better to add a error handling for missing enum values so that in future a new enum value on a updated elastic instance doesn't break existing applications?

[russcam]

@MartinDemberger our general recommendation is to update the client to new minor or patch versions within the a major version lineage, to benefit from bug fixes and other improvements, as well as new APIs if updating the version of Elasticsearch too.

I did consider adding specific error handling for enum deserialization, and introducing an Unknown member, but it didn't feel like the right thing to do - it's true that it may avoid a client call from failing, but if you're relying on a role that ends up being deserialized to Unknown, it doesn't help with this, and ends up masking where something is missing in the client. We have had a discussion about whether node roles should be modelled as an enum, and I would be leaning on modelling them as strings in future, but it's not a change that we would be able to make until 8.0 because of Semantic versioning and binary compatibility. Thankfully, new node roles don't appear often 🙂