Skip to content

fix(fluid-static):Added userData in member connection details #24485

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

neerajcharokar
Copy link
Contributor

@neerajcharokar neerajcharokar commented Apr 30, 2025

Description

We discovered a minor bug that in the Audience service getMembers, user data within connections associated with same userID was being lost when a user with the same userId joined the session multiple times i.e. having multiple connections with different clientId.
This was because only clientId and mode were being stored in each connection.

This PR addresses the issue by adding a userData field alongside clientId and mode in each connection entry, ensuring that user-specific data is accurately preserved for every individual connection.

charokn and others added 2 commits April 30, 2025 19:37
Sure! Here's a clearer and more polished rephrasing:

Added `userData` to each user's `connections` to preserve client-specific details. Previously, only `id` and `mode` were stored, which led to data loss when multiple clients shared the same `userId` but had different associated data.
@github-actions github-actions bot added area: framework Framework is a tag for issues involving the developer framework. Eg Aqueduct base: main PRs targeted against main branch labels Apr 30, 2025
@neerajcharokar neerajcharokar changed the title fix(fluid-static):Added userData in each member connection details fix(fluid-static):Added userData in member connection details Apr 30, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: framework Framework is a tag for issues involving the developer framework. Eg Aqueduct base: main PRs targeted against main branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants