docs icon indicating copy to clipboard operation
docs copied to clipboard

authUrls containing queryString params

Open SimonWoolf opened this issue 8 years ago • 0 comments

cf https://support.ably.io/helpdesk/tickets/1901

We should be much clearer in the docs than we currently are that the authUrl must not contain queryString params, and those should be passed in separately as authParams.

We may also want to put some more validation in the client libraries for this. Currently, ably-js is fairly lenient about this, if you have an authUrl containing some params those'll be used if you don't also specify some authParams. ably-java is a lot stricter; any params in the authUrl are discarded (currently silently -- would be nice if that printed a warning at least)

In future versions of the spec we might also want to standardise that behaviour. Not sure whether we should standardise on leniency (i.e. parse any querystring in the authUrl and either merge with authParams) or strictness. If strictness, that's a breaking change for ably-js, so will need to wait till 2.0.

┆Issue is synchronized with this Jira Task by Unito

SimonWoolf avatar Jun 01 '17 21:06 SimonWoolf