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, Nov 23, 12:35 PM

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

	Unknown Object (File)
	Thu, Nov 14, 8:01 PM

	Unknown Object (File)
	Mon, Nov 11, 2:19 PM

	Unknown Object (File)
	Wed, Nov 6, 7:02 PM

	Unknown Object (File)
	Oct 16 2024, 11:20 PM

	Unknown Object (File)
	Oct 15 2024, 10:49 PM

	Unknown Object (File)
	Oct 9 2024, 8:49 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