Closed yassenb closed 1 year ago
Hey, thanks for this but not going to merge this because elasticsearch actually has some binary APIs: for example, https://www.elastic.co/guide/en/elasticsearch/reference/current/search-vector-tile-api.html
I'm actually planning to use that one and I want to keep this code a bit generalized. Not every REST API returns text.
Can you do a separate PR for the extraParameters fix? That seems useful. Probably there are some more APIs that could benefit from that.
Hey, thanks for this but not going to merge this because elasticsearch actually has some binary APIs: for example, https://www.elastic.co/guide/en/elasticsearch/reference/current/search-vector-tile-api.html
I'm actually planning to use that one and I want to keep this code a bit generalized. Not every REST API returns text.
Can you do a separate PR for the extraParameters fix? That seems useful. Probably there are some more APIs that could benefit from that.