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

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

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

	Unknown Object (File)
	Thu, Dec 12, 10:08 AM

	Unknown Object (File)
	Fri, Dec 6, 8:46 AM

	Unknown Object (File)
	Tue, Dec 3, 8:25 AM

	Unknown Object (File)
	Nov 27 2024, 2:10 PM

	Unknown Object (File)
	Nov 23 2024, 12:35 PM

	Unknown Object (File)
	Nov 19 2024, 10:54 AM

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