Skip to content

Plugin registry checks upon start#3430

Merged
StephenDaDev merged 3 commits into
developmentfrom
users/martinbndr/registry_only_restriction
Jul 12, 2026
Merged

Plugin registry checks upon start#3430
StephenDaDev merged 3 commits into
developmentfrom
users/martinbndr/registry_only_restriction

Conversation

@martinbndr

Copy link
Copy Markdown
Contributor

Plugins will be checked upon start if they are not in the registry and REGISTRY_PLUGINS_ONLY is set to Yes they will be removed. Also plugins are allowed to be removed regardless of being in the registry or not.

Plugins will be checked upon start if they are not in the registry and REGISTRY_PLUGINS_ONLY is set to Yes they will be removed. Also plugins are allowed to be removed regardless of being in the registry or not.
@martinbndr

Copy link
Copy Markdown
Contributor Author

Useful for:

  • Plugins being removed from the registry (for example due to abuse)
  • Hosters that run Modmail for other users and let them provide their DB. They could manually add the plugin to the db and the bot will install regardless of the REGISTRY_PLUGINS_ONLY .env var.

lorenzo132
lorenzo132 previously approved these changes Jan 3, 2026

@sebkuip sebkuip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A small change request for readability

Comment thread cogs/plugins.py Outdated
Fixes the readability of the if statements (combined together with and condition)
@martinbndr martinbndr requested review from lorenzo132 and sebkuip March 7, 2026 18:20

@sebkuip sebkuip left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@StephenDaDev StephenDaDev left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good.

@lorenzo132 lorenzo132 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm.

@StephenDaDev StephenDaDev merged commit 1da6d46 into development Jul 12, 2026
4 checks passed
@StephenDaDev StephenDaDev deleted the users/martinbndr/registry_only_restriction branch July 12, 2026 17:42
@StephenDaDev StephenDaDev added the changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver. label Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog Changes in PR have been added to draft release that will be used for the changelog on the next ver.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants