Skip to content

Commit

Permalink
Change email program from PHP to Sendmail
Browse files Browse the repository at this point in the history
PHP seems to be no longer supported in Nextcloud 14.
This modification might need further testing in different environments.
  • Loading branch information
TomTurnschuh authored and nachoparker committed Sep 16, 2018
1 parent 5be7866 commit 241f2e0
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 2 deletions.
4 changes: 3 additions & 1 deletion changelog.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,7 @@

[v0.59.11](https://github.com/nextcloud/nextcloudpi/commit/701dcd6) (2018-09-16) lamp: add referrer policy for enhanced privacy
[v0.59.12](https://github.com/nextcloud/nextcloudpi/commit/5fc8986) (2018-09-07) Change email program from PHP to Sendmail

[v0.59.11](https://github.com/nextcloud/nextcloudpi/commit/5be7866) (2018-09-16) lamp: add referrer policy for enhanced privacy

[v0.59.10](https://github.com/nextcloud/nextcloudpi/commit/fcbd661) (2018-09-04) ncp-web: add hover text for ncp admin header icons

Expand Down
2 changes: 1 addition & 1 deletion etc/ncp-config.d/nc-init.sh
Original file line number Diff line number Diff line change
Expand Up @@ -111,7 +111,7 @@ EOF
sudo -u www-data php occ config:system:set trusted_domains 5 --value="nextcloudpi.local"

# email
sudo -u www-data php occ config:system:set mail_smtpmode --value="php"
sudo -u www-data php occ config:system:set mail_smtpmode --value="sendmail"
sudo -u www-data php occ config:system:set mail_smtpauthtype --value="LOGIN"
sudo -u www-data php occ config:system:set mail_from_address --value="admin"
sudo -u www-data php occ config:system:set mail_domain --value="ownyourbits.com"
Expand Down
3 changes: 3 additions & 0 deletions update.sh
Original file line number Diff line number Diff line change
Expand Up @@ -144,6 +144,9 @@ chmod 770 /var/www/ncp-web
EOF
}

# NC14 doesnt support php mail
sudo -u www-data php /var/www/nextcloud/occ config:system:set mail_smtpmode --value="sendmail"

} # end - only live updates

exit 0
Expand Down

2 comments on commit 241f2e0

@TomTurnschuh
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd say, an existing SMTP value should not be overwritten, see #637.

@nachoparker
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch!

Please sign in to comment.