New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Need to define SMTP_PORT in conf.php? #397

Closed
funkydan2 opened this Issue Aug 4, 2017 · 4 comments

Comments

Projects
None yet
3 participants
@funkydan2
Contributor

funkydan2 commented Aug 4, 2017

With @s4069b having worked on a roster reminder email script, I've had a play with getting automated roster reminders sent.

For my mail server (Zoho) I wasn't able to connect to the mail server without adding the line
define (SMTP_PORT, '465');
to conf.php.

However, this isn't in the sample configuration file and I also couldn't see where you could set the SMTP port in the web interface.

Not sure if this is a bug, or I've missed the setting somewhere else.

@s4069b

This comment has been minimized.

Show comment
Hide comment
@s4069b

s4069b Aug 4, 2017

Contributor

yes I forgot to mention that. My understanding is that swiftmailer accepts that variable, but it happens not to be specified in the config file.
i.e. I think it's just something missed in the config file.
In my case the server blocks all ports that would work for smtp hence the php mail() alternative for the roster reminder script.

Contributor

s4069b commented Aug 4, 2017

yes I forgot to mention that. My understanding is that swiftmailer accepts that variable, but it happens not to be specified in the config file.
i.e. I think it's just something missed in the config file.
In my case the server blocks all ports that would work for smtp hence the php mail() alternative for the roster reminder script.

@funkydan2

This comment has been minimized.

Show comment
Hide comment
@funkydan2

funkydan2 Aug 4, 2017

Contributor

...and this variable is also required for the email_report.php script to work.

Contributor

funkydan2 commented Aug 4, 2017

...and this variable is also required for the email_report.php script to work.

@s4069b

This comment has been minimized.

Show comment
Hide comment
@s4069b

s4069b Aug 4, 2017

Contributor

yes and the date_reminder script and any other jethro emails
if it's added in conf.php they will all use it
Just means you want to set all email settings in the config file and not under admin system settings in jethro.

Contributor

s4069b commented Aug 4, 2017

yes and the date_reminder script and any other jethro emails
if it's added in conf.php they will all use it
Just means you want to set all email settings in the config file and not under admin system settings in jethro.

@tbar0970

This comment has been minimized.

Show comment
Hide comment
@tbar0970

tbar0970 Aug 5, 2017

Owner

Pull request has fixed the sample config file.

We should also add it to the DB to make it editable in the system config page.

Owner

tbar0970 commented Aug 5, 2017

Pull request has fixed the sample config file.

We should also add it to the DB to make it editable in the system config page.

@tbar0970 tbar0970 added the bug label Aug 5, 2017

@tbar0970 tbar0970 closed this in e4bd7ca Aug 7, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment