Skip to content

Conversation

@microstudi
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings December 19, 2025 20:48
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR adds an email interceptor functionality to prevent emails from being sent to unintended recipients in non-production environments. It allows administrators to configure a list of allowed email addresses or domains, and any outgoing emails will be filtered to only include recipients that match this allowed list.

Key Changes:

  • Implemented a mail interceptor that filters outgoing emails based on an allowed recipients list configured via environment variable
  • Added support for both exact email matching and domain-based pattern matching (e.g., @pokecode.net)
  • Included comprehensive test coverage for various filtering scenarios including case-insensitive matching

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
app/mailers/decidim/pokecode/mail_interceptor.rb Core implementation of the mail interceptor with filtering logic for to/cc/bcc recipients
lib/decidim/pokecode/configuration.rb Adds configuration accessor for ALLOWED_RECIPIENTS environment variable and helper method to parse the list
lib/decidim/pokecode/engine.rb Registers the mail interceptor during Rails initialization when allowed recipients are configured
spec/lib/mail_interceptor_spec.rb Comprehensive test suite covering domain matching, exact email matching, case sensitivity, and mixed recipient scenarios
README.md Documents the new ALLOWED_RECIPIENTS environment variable with usage examples

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

microstudi and others added 6 commits December 19, 2025 22:04
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@microstudi microstudi merged commit 9e3e614 into main Dec 22, 2025
5 of 9 checks passed
@microstudi microstudi deleted the email-interceptor branch January 21, 2026 16:18
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.

2 participants