Fix PHPMailer/SMTP configuration typo for legacy SMTP configurations
ClosedPublic
Actions

Authored by epriestley on Feb 11 2018, 12:04 AM.

Tags

None

Referenced Files

	F15315120: D19058.diff
	Thu, Mar 6, 5:33 PM

	Unknown Object (File)
	Thu, Feb 27, 8:29 PM

	Unknown Object (File)
	Mon, Feb 24, 6:27 PM

	Unknown Object (File)
	Jan 24 2025, 11:21 PM

	Unknown Object (File)
	Jan 23 2025, 8:06 PM

	Unknown Object (File)
	Jan 17 2025, 4:23 PM

	Unknown Object (File)
	Dec 21 2024, 10:52 PM

	Unknown Object (File)
	Dec 13 2024, 10:12 PM

View All Files

Subscribers

None

Details

Reviewers: None
Commits: rPc56bdc92a2cf: (stable) Fix PHPMailer/SMTP configuration typo for legacy SMTP configurations
rPd47c5de9d0ae: Fix PHPMailer/SMTP configuration typo for legacy SMTP configurations

Summary

See https://discourse.phabricator-community.org/t/phpmailer-cannot-read-configuration-value-because-of-typo/1094.

Test Plan

Browsed a dictionary. (This doesn't get hit when configured via cluster.mailers, which is how I previously re-tested SMTP configuration.)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Feb 11 2018, 12:04 AM

Harbormaster completed remote builds in B19488: Diff 45686.Feb 11 2018, 12:05 AM

This revision was not accepted when it landed; it landed in state Needs Review.Feb 11 2018, 12:05 AM

epriestley requested review of this revision.

Closed by commit rPd47c5de9d0ae: Fix PHPMailer/SMTP configuration typo for legacy SMTP configurations (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

metamta/

adapter/

PhabricatorMailImplementationPHPMailerAdapter.php

2 lines

Diff 45687

View Options

src/applications/metamta/adapter/PhabricatorMailImplementationPHPMailerAdapter.php

Fix PHPMailer/SMTP configuration typo for legacy SMTP configurationsClosedPublicActions