milli
milli copied to clipboard
`validate_document_id` function trims the id, but maybe shouldn't
See: https://github.com/meilisearch/milli/pull/561#discussion_r925394083
So a document such as:
{
"id" : " hello "
}
will have the id:
"hello"
but according to the Meilisearch documentation, spaces are not allowed in document ids. So one could argue that the document should not be accepted in the first place.
And thus the documents
{ "id": " hello", ... }
{ "id": "hello ", ... }
Will be merged together, which is clearly a bug
I've created https://github.com/meilisearch/meilisearch/issues/2640 to track it for the sprint. Feel free to close this issue in favor of this one if you prefer
Closed in favor of https://github.com/meilisearch/meilisearch/issues/2640