Skip to content

Conversation

@icewind1991
Copy link
Member

Since guest are identified by their email, allowing users to change them is probably a bad idea.

@icewind1991 icewind1991 added this to the Nextcloud 32 milestone Aug 6, 2025
@icewind1991 icewind1991 added the 3. to review Waiting for reviews label Aug 6, 2025
@icewind1991 icewind1991 force-pushed the guest-no-email-change branch from 829830e to cccc4b9 Compare August 6, 2025 14:52
@icewind1991 icewind1991 requested review from Pytal and skjnldsv and removed request for Pytal August 6, 2025 14:52
@icewind1991
Copy link
Member Author

And a repair step to fix cases where this was already changed: #1370

@icewind1991 icewind1991 force-pushed the guest-no-email-change branch from cccc4b9 to deafd78 Compare August 6, 2025 15:12
@icewind1991 icewind1991 merged commit 72c4439 into main Aug 11, 2025
45 checks passed
@icewind1991 icewind1991 deleted the guest-no-email-change branch August 11, 2025 08:25
@skjnldsv skjnldsv mentioned this pull request Aug 12, 2025
@nc-enterprise-user
Copy link

Please forgive me if this is not the right place to comment but this change has significant impact.

It is true that Guests are identified by their email, but it's also very common and natural that a user needs to update their email. This would then force any Guest email change to someone with administrative permissions and that is a lot of workload for networks with many Guest users.

Prior to this update, a Guest could update their own email and log in as either the initial email that was added, or the "primary" email in their profile.

I believe it is a significant functionality loss to not allow Guest users to be able to manage their own email and I appreciate you reading this view point. I am happy to provide more information if that is helpful.

Is there a conversation that discussed the primary concerns of Guests being able to change their emails?

@icewind1991
Copy link
Member Author

#1379 adds a config flag

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

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants