echoMessages default does not send echo=true query parameter (RTC1a)

[paddybyers]

Spec requirement

RTC1a: The echoMessages option (default true) should be sent as an echo=true query parameter on the WebSocket connection URL.

Current behaviour

ably-js only sends echo=false when echoMessages is explicitly set to false. When echoMessages is true (the default), no echo parameter is sent — the server defaults to echoing.

Reproduction

RUN_DEVIATIONS=1 npx mocha --no-config --require tsx/cjs 'test/uts/realtime/client/realtime_client.test.ts' --timeout 30000 --grep "RTC1a - echoMessages default sends echo=true"

UTS test

test/uts/realtime/client/realtime_client.test.ts — asserts the WebSocket URL contains echo=true.

┆Issue is synchronized with this Jira Task by Unito

[SimonWoolf]

...Valid bug according to the current spec, but I don't know why the current spec says this, looks like it was a recent change to make the qs param required, but that PR doesn't give a reason for that behaviour change, @lawrence-forooghian ?

[paddybyers]

I agree, I think this is over-zealousness in the spec. I've raised ably/specification#465 but will leave this open until that's resolved

[lawrence-forooghian]

Replied on ably/specification#198 and fixed in ably/specification#469