-
Notifications
You must be signed in to change notification settings - Fork 55
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
clientId from token auth is ignored for presence #198
Labels
bug
Something isn't working. It's clear that this does need to be fixed.
Comments
mattheworiordan
added
bug
Something isn't working. It's clear that this does need to be fixed.
high priority
labels
Jan 13, 2016
I don't think it's related to #195 actually, I configured the library to only use websockets, and I still get the error:
|
Ah. All the clientid inheritance tests we added were checking that auth.clientId was set correctly, I never thought to check that auth.clientId was where realtimepresence was actually getting its clientId from 😞 PR #199 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Here is some code to demonstrate the bug:
The URL
/realtime-token
returns the following token request:The error in the logs is:
It seems the Ably client library is not picking up the
clientId
from theConnectionDetails
correctly, although this may simply be caused by #195 so this can be ignored if that is the case. Feel free to close if it is definitely caused by that issue, which I expect it is.Full log from console:
The text was updated successfully, but these errors were encountered: