Skip to content

Conversation

@alankessler
Copy link

@RocketChat/core

Right now there is no way to hide the user list from low-permission users. This should allow the administrator to disable the autocomplete feature.

@graywolf336
Copy link
Contributor

What happens to the user list on the side panel?

@alankessler
Copy link
Author

alankessler commented Apr 25, 2017

@graywolf336 Probably what you expected. The guest user appears online, but all other users in the channel appear offline to the guest.

image

image

@alankessler
Copy link
Author

But it does fix the problem I'm concerned about:

as guest:
image

as admin:

image

@Hudell
Copy link
Contributor

Hudell commented Nov 7, 2018

I wonder why was this never merged?

@alankessler do you mind fixing the conflicts on the PR?

@RocketChat RocketChat deleted a comment Nov 8, 2018
@RocketChat RocketChat deleted a comment Nov 8, 2018
@RocketChat RocketChat deleted a comment Nov 8, 2018
@alankessler
Copy link
Author

I wonder why was this never merged?

@alankessler do you mind fixing the conflicts on the PR?

I think this is fixed.

@rodrigok
Copy link
Member

We are still missing a permission to list the members of rooms, I think we should add this new permission.

@rodrigok rodrigok added this to the 3.4.0 milestone May 28, 2020
@rodrigok rodrigok changed the title [NEW] Permissions setting to deny low-privilege users access to the user au… [NEW] Permissions setting to deny low-privilege users access to the user autocomplete list Jun 19, 2020
@rodrigok rodrigok modified the milestones: 3.4.0, 3.5.0 Jun 19, 2020
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ alankessler
❌ Alan Kessler


Alan Kessler seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants