[BUG] Url serialize Option<Vec<String>>

elastic / elasticsearch-rs

Official Elasticsearch Rust Client

https://www.elastic.co/guide/en/elasticsearch/client/rust-api/current/index.html

Apache License 2.0

12 stars 73 forks source link

[BUG] Url serialize Option<Vec<String>> #12

Closed russcam closed 4 years ago

russcam commented 4 years ago

Serde's UrlEncodedSerializer<Target> does not know how to serialize an Option<Vec<String>> with some value. This should be serialized as one value, where the vec values are joined by commas.

russcam commented 4 years ago

Opened #13 to address