Skip to content

Conversation

@gbp
Copy link
Member

@gbp gbp commented Jan 28, 2026

Relevant issue(s)

Fixes #8801

What does this do?

Refactor the mail ingression pipeline, removing source restrictions.
Adds the ability to receive email via ActionMailbox.

Why was this needed?

Implementation notes

Screenshots

Notes to reviewer

Letting CI run to find broken tests. Could split out into two PRs, 1 for the refactoring and 2 adding ActionMailbox.


[skip changelog]

@gbp gbp force-pushed the add-action-mailbox branch 2 times, most recently from 0c5e86a to 41f6816 Compare January 29, 2026 09:23
gbp added 6 commits February 2, 2026 15:56
Integrate ActionMailbox into `RequestMailer.receive` for inbound email
handling. This maintains existing functionality but lays the groundwork
for future enhancements.

Benefits of using ActionMailbox include:

1. Enhanced email routing capabilities, enabling specialised
   processing like the Excel hidden data spreadsheet analyzer.
2. Refactoring opportunities for `RequestMailer#receive`, particularly
   for spam detection, duplicate email handling, and initial request
   assessment.
3. Clear separation of concerns between Mailers (for sending) and
   Mailboxes (for receiving).
4. Improved email processing efficiency through ActionMailbox,
   facilitating background job handling and potential simplification
   of mail ingress.
5. Provides a solution for re-users to receive emails without needing
   their own mail server setup.
Include options for ActionMailbox supported services in the retriever
method and set the ingress setting.
Instead of calling inline extract into a method so we can call later
in the install script after bundle install has been run.

This is so we can set the ingress password in the Rails credentials
needed for the ActionMailbox migration.
Set credentials and correctly configure Postfix pipe to use
ActionMailbox ingress with the correct password.
Ensure encrypted credentials and private key aren't committed into the
repo.

Normally for Rails it is acceptable for the encrypted credentials to be
committed but given Alaveteli powers multiple sites we don't want this.
@gbp gbp force-pushed the add-action-mailbox branch from 41f6816 to 314ddff Compare February 2, 2026 16:50
@gbp gbp changed the base branch from develop to add-action-mailbox-prep February 2, 2026 16:51
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.

Integrate ActionMailbox

3 participants