Getting E-Mails from inactive users

I’m using incoming mail support for group messaging with staged users.

This works fine if the address the message is coming from is new. However, I am re-purposing a Discourse installation for a yearly event, and some old users are stuck with an inactive accounts they can no longer log in to.

When such a user sends a mail to Discourse, they are rejected with Email::Receiver::InactiveUserError.
What’s the best method to prevent this issue? Would replacing the mail address of the inactive accounts work?

إعجاب واحد (1)

Any thoughts here @zogstrip?

إعجاب واحد (1)

The InactiveUserError happens only when the user is not activated and not staged. If you want your users to be able to send emails in, you will have to either activate their account or mark them as staged.

إعجابَين (2)

Can I mark existing, inactive users as staged? Would that have any side effects?

Other than enabling them to send emails in, none :wink:

إعجاب واحد (1)

How can I do that, then? The admin UI does not offer this option.

Will they return to normal if they ever log in?

You’ll have to use the rails console or run a SQL query.

Yes.

إعجاب واحد (1)

Hm, it looks like this doesn’t work.

I had a non-active, staged user (that had been disabled via the interface and staged via Rails). After the user logged in (via SSO), he had to confirm has address, became active and could log in, but is still staged:

إعجاب واحد (1)

Hmm, it might not unstage when using SSO. Having a look.

إعجابَين (2)

Thanks a lot!

In the meantime, for anyone who wants to try the same approach, here’s what I ran on the rails console (rails c inside the container):

User.where({ active: false }).update_all({ staged: true })
إعجاب واحد (1)

You can do slightly shorter :wink:

User.where(active: false).update_all(staged: true)
إعجابَين (2)

I’ve given up understanding Ruby’s syntax a long time ago – even my version took many tries to be correct. Thanks for the shorter version :wink:

Just pushed a fix :helicopter:

https://github.com/discourse/discourse/commit/874c18cbc1c4768412cfe9ca08f416295f112c8b

4 إعجابات

After a journey into tests-passed lands, I can confirm this is fixed. Thanks a lot!

3 إعجابات