Getting E-Mails from inactive users

(Felix Freiberger) #1

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?

Re-purposing a Discourse installation for a yearly event
(Jeff Atwood) #2

Any thoughts here @zogstrip?

(Régis Hanol) #3

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.

(Felix Freiberger) #4

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

(Régis Hanol) #5

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

(Felix Freiberger) #6

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

Will they return to normal if they ever log in?

(Régis Hanol) #7

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


(Felix Freiberger) #8

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:

(Régis Hanol) #9

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

(Felix Freiberger) #10

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 })

(Régis Hanol) #11

You can do slightly shorter :wink:

User.where(active: false).update_all(staged: true)

(Felix Freiberger) #12

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:

(Régis Hanol) #13

Just pushed a fix :helicopter:

(Felix Freiberger) #14

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

Inactive user error for an active user?
(Jeff Atwood) closed #15