Skip to content

Add blocklist functionality to commentbot and WebUI#481

Open
rayegun wants to merge 7 commits intomasterfrom
blocklist
Open

Add blocklist functionality to commentbot and WebUI#481
rayegun wants to merge 7 commits intomasterfrom
blocklist

Conversation

@rayegun
Copy link
Copy Markdown

@rayegun rayegun commented Apr 3, 2026

Heavily vibe-coded by Claude Opus 4.6, but examined line by line manually.

Intended to resolve #443 by adding a simple configurable blocklist check to both the commentbot and the webui. The blocklist is a (private) repo with a TOML file, and the commentbot will now fetch and check user IDs (not usernames) against this file. There is a configurable cache time on reloading the TOML file to avoid spamming the GitHub API and reduce latency on registration calls.

Should fail gracefully allowing all registrations through.

Replaces #480 in tree.

@rayegun rayegun mentioned this pull request Apr 3, 2026
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@rayegun
Copy link
Copy Markdown
Author

rayegun commented Apr 3, 2026

Alright should be ready. You'll need to add the correct configs to the app and I suppose the WebUI, they are documented in the existing config files.

Can you test with your bot once it's merged?

@DilumAluthge
Copy link
Copy Markdown
Member

Can you test with your bot once it's merged?

It won't take effect immediately after this PR is merged. We'll first need to register a new release of Registrator.jl. But after that, we need to ping the appropriate folks in the JuliaLang Slack1, and then those folks would update the production instance of JuliaRegistrator.

Footnotes

  1. I usually ping Luke in the #pkg-registration channel of the JuliaLang Slack.

@DilumAluthge
Copy link
Copy Markdown
Member

CC: @nkottary @lamdor So that you know what's coming down the pipeline.

Copy link
Copy Markdown
Member

@nkottary nkottary left a comment

Choose a reason for hiding this comment

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

LGTM

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.

add mechanism to block users

3 participants