Martin Brennan

Just noting that at some point in the future I would be interested in support for resumable S3 multipart uploads as well!

I will uncomment all the other test runs before merging this.

> Maybe we move allowPersonalMessages to current user service, in your code it's only code using current user as a param afaict. This way we can compute on current user...

Spec failures unrelated

**NOTE**: I need a migration here to set the `outbound_message_id` for existing posts which have an associated `IncomingEmail` record.

Hi @evgeni sorry for the delay in reviewing here. I am not sure what the benefit of doing this would be since everything within the discourse Email::Receiver flow assumes the...

> Which will allow admins to actually see those bounces in the interface, versus going hunting down logs somewhere on disk. Plus, it eliminates the weird "internal API error" reject...

Just merged main into this so it's not too far out of date and used `DMenu` for the bookmark-menu just to get it working though looking crap. Will keep working...

@jjaffeux IMO if you approve and merge this it would be better, then we can split up and work on various parts of it and not end up with a...

Just echoing what @jbcpollak is saying above. I am an engineer at Discourse and we are tracking this in https://meta.discourse.org/t/add-richer-authentication-support-for-pop3/219340 and we definitely have a need for POP3 OAUTH2 support...