FriendsOfFlarum Subscribed

Leanna The errors are telling you that your SMTP (outgoing mail) connection was not properly authenticated by your email server.

The two most common causes of this error are:

• Your host requires checking on all outbound email. (Checking email first is one way your email provider manages the security of your email account.)
• Your installation of Flarum contains erroneous mail details.

Are you able to receive emails on signing signing up to your forum?

Hi, sorry for bugging you, but can someone please describe exactly to which events are these subscriptions? Thanks in advance!

EricEndo my apologies, I was to provide some screenshots today but I became VERY busy and didn't have time.

The two event notification types available are for when a new user joins and when a user creates a new discussion.

katos I can sign my mail normally.

Hi @EricEndo and to those who are wondering what this extension actually does, I added some screenshots to my first post just to clear any concerns and mystery

Also, feel free to suggest any other events you think would be good candidates for this extension!

The localisation file of this extension mentions two options I can't find anywhere:

defaults_label: Set "{setting}" enabled by default
forced_label: Force enable "{setting}" for everyone

Is this a planned feature or just hidden well enough?

Ralkage Is the problem I mentioned solved?

Pollux possibly a planned feature this is continued work and I mainly just worked on completing the implementation of the initial features/events as well as the update to Beta 9.

Leanna you seem to be the only one having this issue and @katos previously replied to it. Unless someone else reports something similar, even I myself cannot reproduce your issue.

ctx That's not good . Looks like this code is the issue, there was never an actual subject set for the email it seems:

    /**
     * Get the subject line for a notification email.
     *
     * @return string
     */
    public function getEmailSubject()
    {
        return $this->user;
    }

datitisev whoops

0.1.1

This security related bug fix should stop user information (such as passwords and email addresses) from being added to the email subject's meta info which was reported here https://discuss.flarum.org/d/20917-friendsofflarum-subscribed/24.

For those of you that have this extension installed, it is advised that you update it ASAP.

Updating

composer require fof/subscribed

Ralkage 0.1.1
This security related bug fix should stop user information (such as passwords and email addresses) from being added to the email subject

Thank you! I have re-tested and can confirm this issue has been resolved with v0.1.1 on Flarum v0.1.0-beta.10

Greetings. On your next development pass, please consider updating the subject line for new user subscriptions to include the forum name. This may help those who maintain multiple-forums quickly know where the new user request came from. Understood, the forum name is included in the body, but having it in the subject too would be icing on the cake.

Currently: [New User] CrashTestDummy
Proposed: [Oakhurst at Kingwood] New User CrashTestDummy

Thank you for everything that you do.

Respectfully submitted, Eric

ctx does the following subject line work?

Proposed: [Forum Name] New User Registration

This should keep things generic (something I think WordPress does?) and simple, leaving the body of the email to include additional useful information and links.

We can also try for:

• [Forum Name] New User - ExampleUsername
• [Forum Name] New User Registration - ExampleUsername
• [Forum Name] New User Just Registered - ExampleUsername

Others that use this extension as well, please let me hear your thoughts! I am considering adding more translations in the future, especially for the email templates so that you can freely edit with fof/linguist (if at all possible ).

Ralkage Proposed: [Forum Name] New User Registration

I think that is acceptable. I don't necessarily wish to know "Who" registered. Knowing which forum is a big help while administrating several. Thanks for the consideration and feedback!

Hello, it could be more usefull to be noticed that a moderator or an admin has modified your post for example.

Ralkage Others that use this extension as well, please let me hear your thoughts! I am considering adding more translations in the future, especially for the email templates so that you can freely edit with fof/linguist (if at all possible ).

What about providing a template system, so admins can freely chose a text for the subject and the body using placeholders like [username], [email], [time] etc. So it would be up to the admin to choose the information provided in the subject line and in the body of the e-mail.

This approach would also provide the necessary means for translations. Of course the extension could offer suggestions for the wording in different languages, but your own phrases will always be possible, so missing translations wouldn't matter a lot.