-
Notifications
You must be signed in to change notification settings - Fork 1.3k
ReactiveElasticsearchOperations indexName twice endcoding #1665
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
Comments
"<foo-{now/M{yyyy.MM}}>" is not a valid index name. The index name must be either a lowercases String or a SpEL (Spring Expression language) template expression. What do you want to achieve with the String you provide? Check https://www.elastic.co/guide/en/elasticsearch/reference/current/indices-create-index.html#indices-create-api-path-params for what characters are allowd in an index name. |
i want Date math support in index names https://www.elastic.co/guide/en/elasticsearch/reference/current/date-math-index-names.html |
Ok; didn't know that. I will need to investigate this and to try to reproduce it. |
ok, I can reproduce that. That reactive client escapes the '%' with '%25'. This does not happen in the non-reactive part. |
implemented in the current master and backported to 4.1.x and 4.0.x |
@sothawo good! Thank you |
this example request
webClient Request url
Expecting :
/%3Cfoo-%7Bnow%2FM%7Byyyy.MM%7D%7D%3E/_doc
but :
/%253Cfoo-%257Bnow%252FM%257Byyyy.MM%257D%257D%253E/_doc
elasticsearch error log
reason=Invalid index name [%3Cfoo-%7Bnow%2FM%7Byyyy.MM%7D%7D%3E], must be lowercase]
The text was updated successfully, but these errors were encountered: