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.

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

Audit registrations #7

Merged
merged 3 commits into from
Nov 4, 2024

Conversation

AndrewKvalheim
Copy link
Member

Not really sure if we’d want this, but exploring the idea. Now that we’re administering Synapse we have the ability to add custom Synapse modules.

In past years we’ve been blind to actual user registrations and relied upon observing room membership as a proxy, but that risks missing problems like conference attendees failing to join the space or deliberate abuse. As a simple mitigation, this hooks into on_user_registration and logs each new user in a moderation room for human review.

@strugee
Copy link
Member

strugee commented Nov 1, 2024

I'd like this to go to a room on the ephemeral homeserver, not our production space. Rationale: if like 10,000 bots sign up or whatever, we've just permanently DoS'd Seattle Matrix's database.

@AndrewKvalheim
Copy link
Member Author

@altsalt ^ Recommendations on where to send this?

@altsalt
Copy link
Member

altsalt commented Nov 1, 2024

Perhaps we can setup some sort of a bot log channel (though of course this is more than just bot), where there is an expected large number of items to be continuously coming in.

@altsalt
Copy link
Member

altsalt commented Nov 1, 2024

I mean, given the title, perhaps #audit or #syslog?

AndrewKvalheim added a commit to SeaGL/patch that referenced this pull request Nov 4, 2024
@strugee
Copy link
Member

strugee commented Nov 4, 2024

Does it matter? This is just for SeaGL staff so let's just pick a name that the three of us can live with and get on with it (unless I'm missing something?)

@AndrewKvalheim
Copy link
Member Author

I’ve already created #audit:2024.seagl.org and listed it in the Restricted space, so the only missing piece is adding this module

@altsalt
Copy link
Member

altsalt commented Nov 4, 2024

It's interesting, some of what gets dumped into Triage is also the content I'd expect in Audit. Anyway, think this is all good, is there something I can help with here?

@AndrewKvalheim
Copy link
Member Author

I’d been awaiting an explicit affirmative from anyone else on whether we want to install the module, preferably informed by having read it since I know next to nothing about doing this. But it seems like we all want it and it’s been working in the staging instance.

@AndrewKvalheim AndrewKvalheim merged commit ae6ee4a into seagl-fork Nov 4, 2024
2 of 5 checks passed
@AndrewKvalheim AndrewKvalheim deleted the AndrewKvalheim/audit-registrations branch November 4, 2024 22:52
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