Skip to content
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.

Sign up for GitHub

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

Jump to bottom

User role is incorrect after client reconnect. #11383

Merged
merged 1 commit into from Feb 11, 2021
Merged

User role is incorrect after client reconnect. #11383

merged 1 commit into from Feb 11, 2021

Conversation

aron-2809
Copy link
Contributor

@aron-2809 aron-2809 commented Feb 11, 2021 •
edited by ritzalam

What does this PR do?

Check the resulting role passed through the message body instead of the user's current role.

Closes Issue(s)

#11182

@antobinary antobinary added this to the Release 2.3 milestone Feb 11, 2021
@aron-2809
Copy link
Contributor Author

@ritzalam

@ritzalam
Copy link
Member

To test:

  1. Open 2 browsers and have 2 users join as moderator.
  2. Have one moderator demote the other to viewer.
  3. Copy the viewer's url and close the viewer's browser tab. Look at the moderator's users list. Wait for the viewer to disappear from the user's list.
  4. Open a new browser tab and paste the viewer's url. This simulates a user rejoining the meeting..
  5. The user should still be a viewer not it's original role (moderator)

@ritzalam ritzalam merged commit 71eb8cc into bigbluebutton:develop Feb 11, 2021
@antobinary
Copy link
Member

Is this resolved as well? #10376

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
module: core
Projects
None yet
Milestone
Release 2.3
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@aron-2809 @ritzalam @antobinary