[PR #152] [MERGED] Fix mail config typo in sample config #290

New issue

Closed

opened 2026-02-26 10:30:39 +03:00 by kerem · 0 comments

kerem commented

2026-02-26 10:30:39 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/opensolutions/ViMbAdmin/pull/152
Author: @kaechele
Created: 8/27/2015
Status: ✅ Merged
Merged: 4/2/2016
Merged by: @barryo

Base: master ← Head: patch-1

📝 Commits (1)

235b206 Fix mail config typo in sample config

📊 Changes

1 file changed (+14 additions, -8 deletions)

View changed files

📝 application/configs/application.ini.dist (+14 -8)

📄 Description

For Zend Framework the mail resource is actually configured through resources.mail.transport, not resources.mailer.
Replace the misleading sample configuration with correct examples.
With the erroneous configuration mails would not be sent if the underlying mailserver does strict HELO checks (i.e. reject mails with HELO localhost), leading to error messages about mails not being able to be sent.

This might help fix bug reports such as #25

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/opensolutions/ViMbAdmin/pull/152 **Author:** [@kaechele](https://github.com/kaechele) **Created:** 8/27/2015 **Status:** ✅ Merged **Merged:** 4/2/2016 **Merged by:** [@barryo](https://github.com/barryo) **Base:** `master` ← **Head:** `patch-1` --- ### 📝 Commits (1) - [`235b206`](https://github.com/opensolutions/ViMbAdmin/commit/235b20663fbe2b9eeaccccef06801d19d8aee029) Fix mail config typo in sample config ### 📊 Changes **1 file changed** (+14 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `application/configs/application.ini.dist` (+14 -8) </details> ### 📄 Description For Zend Framework the mail resource is actually configured through resources.mail.transport, not resources.mailer. Replace the misleading sample configuration with correct examples. With the erroneous configuration mails would not be sent if the underlying mailserver does strict HELO checks (i.e. reject mails with HELO localhost), leading to error messages about mails not being able to be sent. This might help fix bug reports such as #25 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>