-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Indicies.Get call fails to deserialize ICollections properly in the IndexRequest #8034
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
Indicies.Get call fails to deserialize ICollections properly in the IndexRequest #8034
Comments
Hey @baveryt, could you please include the JSON payloads for the request (and response, if possible)? They should be included in the exception after calling |
@flobernd - The call actually does not throw an exception, it just reports that it was an invalid request. I'll include what I can here
The response reports back IsValidResponse = false. The Debug information shows:
The ElasticsearchServerError shows:
It's all pointing to the Features list couldn't be converted properly, it should have built the request like: GET /search_%2A?features=aliases You will also see a similar problem if you do something similar to the ExpandWildcards property. |
Thanks for the additional details! It indeed seems like building the query argument fails for some cases when the argument is a collection. |
Elastic.Clients.Elasticsearch version: 8.11
Elasticsearch version: 8.8
.NET runtime version: 8.0
Operating system version: Osx
Description of the problem including expected versus actual behavior:
When making calls to Indicies.Get and setting the Features or ExpandWildcards on the GetIndexRequest object the ICollections are not getting parsed properly for the underlying API call. Looking at the stack trace you'll see something like: Invalid Elasticsearch response built from a unsuccessful (400) low level call on GET: /%2A?features=System.Collections.ObjectModel.Collection%601%5BElastic.Clients.Elasticsearch.IndexManagement.Feature%5D\n
Steps to reproduce:
Expected behavior
For the features it should expand out the collections properly [aliases] in this case.
The text was updated successfully, but these errors were encountered: