Skip to content

Restrict global chat to authenticated users#247

Merged
system2k merged 4 commits into
system2k:masterfrom
ossftw:no-anon-global-chat
Jun 1, 2026
Merged

Restrict global chat to authenticated users#247
system2k merged 4 commits into
system2k:masterfrom
ossftw:no-anon-global-chat

Conversation

@ossftw
Copy link
Copy Markdown
Contributor

@ossftw ossftw commented May 28, 2026

This pull request prevents users without accounts from chatting in the global chat.

  • cuts down on spam and throwaway messages from anonymous users
  • makes moderation significantly easier since every message is tied to an account...
  • improves accountability, so users are less likely to abuse global chat
  • keeps global chat more stable and usable for actual interaction

@PizzaTowerFanGD
Copy link
Copy Markdown

iirc most nwots dont use uvias

@ossftw
Copy link
Copy Markdown
Contributor Author

ossftw commented May 28, 2026

iirc most nwots dont use uvias

if(location == "global" && accountSystem == "uvias") {

@PizzaTowerFanGD
Copy link
Copy Markdown

i think you could just check for realUsername? idk im not sure

@ossftw
Copy link
Copy Markdown
Contributor Author

ossftw commented May 30, 2026 via email

@ossftw
Copy link
Copy Markdown
Contributor Author

ossftw commented May 30, 2026

Everything works as intended on my end.

@ossftw ossftw marked this pull request as draft May 31, 2026 13:40
@ossftw ossftw changed the title Restrict global chat to authenticated Uvias users Restrict global chat to authenticated users May 31, 2026
Copy link
Copy Markdown
Contributor Author

@ossftw ossftw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • per-world option to enable/disable anonymous chats
  • admin-panel switch for Global chat allowing anonymous chats

@ossftw ossftw marked this pull request as ready for review May 31, 2026 14:30
@system2k system2k merged commit 8719ca7 into system2k:master Jun 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants