queryUnverified does not thread desiredHeight

[kriskowal]

In the Stargate QueryClient, the method queryUnverified does not thread an optional third argument for the height of the query. This is inconsistent with queryVerified. Is this omission intentional?

cosmjs/packages/stargate/src/queryclient/queryclient.ts

Lines 573 to 585 in 155ed33

	public async queryUnverified(path: string, request: Uint8Array): Promise<Uint8Array> {
	const response = await this.tmClient.abciQuery({
	path: path,
	data: request,
	prove: false,
	});
	if (response.code) {
	throw new Error(`Query failed with (${response.code}): ${response.log}`);
	}
	return response.value;
	}

[webmaster128]

I think it was just never needed for our use cases. The height in queryVerified is needed for some proof stuff.

[webmaster128]

Can you have a look at #1255?

[webmaster128]

Shipped as part of CosmJS 0.29.0