Skip to content

Comments

feat: add ip address, user agent, referer and session token if available in the UserJoined event#68

Merged
prlanzarin merged 2 commits intobigbluebutton:developfrom
mconf:expanded-join
Oct 24, 2025
Merged

feat: add ip address, user agent, referer and session token if available in the UserJoined event#68
prlanzarin merged 2 commits intobigbluebutton:developfrom
mconf:expanded-join

Conversation

@fcecagno
Copy link
Member

No description provided.

…-joined)

The `ip-address` and `user-agent` fields of `user-joined` have two
issues:
- Incorrect naming: they use `_` instead of `-` as separators; this is
  not consistent with other attributes
- Incorrect parsing: BBB events use `userAgent` and `ipAddress` and
  webhooks is parsing that incorrectly (i.e.: attributes are never sent)

Fix parsing of the above attributes so that they're always sent in
`user-joined` when available. Additionally, use `-` as separators for
the sake of consistency.
Add the `session-token` attribute to `user-joined` event when available.
The payload is an array of strings (session tokens).

Here, it's most likely going to be a 1-sized array for the foreseeable
future since only the first "join" event for an user is handled (i.e.:
no multi-session handling).

Keeping it an array should make things future-proof, though, for when
multi-session scenarios are handled.
@prlanzarin prlanzarin self-requested a review October 23, 2025 14:27
@prlanzarin prlanzarin merged commit 5290c10 into bigbluebutton:develop Oct 24, 2025
3 of 4 checks passed
@prlanzarin prlanzarin deleted the expanded-join branch October 24, 2025 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants