[NO MERGE] Clarify user claim handling for presence and annotations#3200
[NO MERGE] Clarify user claim handling for presence and annotations#3200
Conversation
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
0558b5d to
5072d7c
Compare
Add clarification that custom restrictions also apply to presence messages and individual annotations. The user claim is included in extras.userClaim for both cases (presence.extras.userClaim and annotation.extras.userClaim). https://claude.ai/code/session_01TKV8vHU7MP62PNzDJuKwiF
5072d7c to
17048d1
Compare
|
@ably/team-deved Not ready to merge until next week, but ready for review :) |
m-hulbert
left a comment
There was a problem hiding this comment.
Small suggestion but otherwise LGTM
| Messages can be annotated with trusted metadata copied from the client's authentication token by Ably servers. Clients are unable to directly publish messages with user claim metadata, and claims contained within the authentication token are signed to prevent tampering. Claims can be scoped to individual channels or to namespaces of [channels](/docs/channels). The most specific user claim will be added to the message as part of the `extras` object. Note that this does not apply to presence or metadata messages. | ||
| Messages can be annotated with trusted metadata copied from the client's authentication token by Ably servers. Clients are unable to directly publish messages with user claim metadata, and claims contained within the authentication token are signed to prevent tampering. Claims can be scoped to individual channels or to namespaces of [channels](/docs/channels). The most specific user claim will be added to the message as part of the `extras` object. | ||
|
|
||
| Custom restrictions also apply to presence messages and individual annotations. In both cases, the user claim is included in `extras.userClaim`. |
There was a problem hiding this comment.
Worth a link to presence and annotations pages I think
Description
This PR updates the documentation for JWT user claims in Ably to include Presence and Annotations.
CHA-1243
Checklist
https://claude.ai/code/session_01TKV8vHU7MP62PNzDJuKwiF