Bug: extra space in the call leading to BAD_REQUEST: unable to parse query

[hrkrshnn]

beacon-api-client/src/api_client.rs

Line 152 in cb73ed7

let validator_ids = validator_ids.iter().join(", ");

I think it should be .join(",")

The current implementation produces

Sending: RequestBuilder { method: GET, url: Url { scheme: "http", cannot_be_a_base: false, username: "", password: None, host: Some(Domain("beacon-chain.lighthouse.dappnode")), port: Some(3500), path: "/eth/v1/beacon/states/head/validators", query: Some("id=0x800040778c8f2ef6dc36009975f69a5a3fb6cecbee2fbb0fe5b5a8dc37241fe35e043299cc7c206cc7e522af99cc4a6f%2C+0x8000558843d93fd74ad009d1f120e657c09fea88c44d1d739dccbafdca9e6f5df6d2649825ac7db5f2078aade4bf03cb"), fragment: None }, headers: {} }

-id=0x....99cc4a6f%2C+0x80005
+id=0x....99cc4a6f%2C0x80005

[hrkrshnn]

There are a couple of other places with the same issue. Would be up for submitting a PR.

[ralexstokes]

ah nice find!

made an issue to track adding some tests ralexstokes/ethereum-consensus#237