Skip to content

feat: Hash user IDs by default for improved privacy - #1335

Merged
nickvergessen merged 1 commit into
mainfrom
feat/noid/hash-user-id-for-privacy
Mar 31, 2026
Merged

feat: Hash user IDs by default for improved privacy#1335
nickvergessen merged 1 commit into
mainfrom
feat/noid/hash-user-id-for-privacy

Conversation

@nickvergessen

Copy link
Copy Markdown
Member
  • Enabled by default, but optional
  • Improves privacy of guests as their user id does not automatically leak their email address (e.g. avatars in chats and more)

@nickvergessen nickvergessen added enhancement New feature or request 3. to review Waiting for reviews labels May 1, 2025
@nickvergessen nickvergessen self-assigned this May 1, 2025
Comment thread lib/Command/AddCommand.php
Comment thread lib/UserBackend.php
@nickvergessen
nickvergessen force-pushed the feat/noid/hash-user-id-for-privacy branch from 33df97e to 2576f83 Compare March 27, 2026 07:23
@nickvergessen
nickvergessen requested a review from come-nc March 27, 2026 07:24
@nickvergessen
nickvergessen force-pushed the feat/noid/hash-user-id-for-privacy branch from 2576f83 to d2de5fd Compare March 27, 2026 07:27
@nickvergessen

Copy link
Copy Markdown
Member Author

Rebased as the birthday is approaching. Since it has an appconfig it's also low risk

Comment thread lib/UserBackend.php Outdated
@nickvergessen
nickvergessen force-pushed the feat/noid/hash-user-id-for-privacy branch from d2de5fd to 703a64d Compare March 30, 2026 09:04
@nickvergessen
nickvergessen requested a review from come-nc March 30, 2026 09:04
@nickvergessen

nickvergessen commented Mar 31, 2026

Copy link
Copy Markdown
Member Author

Signed-off-by: Joas Schilling <coding@schilljs.com>
@nickvergessen
nickvergessen enabled auto-merge March 31, 2026 15:12
@nickvergessen
nickvergessen force-pushed the feat/noid/hash-user-id-for-privacy branch from 703a64d to 555cf1b Compare March 31, 2026 15:13
@nickvergessen
nickvergessen merged commit cd26d8a into main Mar 31, 2026
54 checks passed
@nickvergessen
nickvergessen deleted the feat/noid/hash-user-id-for-privacy branch March 31, 2026 15:17
@icewind1991

icewind1991 commented Apr 29, 2026

Copy link
Copy Markdown
Member

This breaks login #1542

@salmart-dev

This comment was marked as resolved.

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 enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants