Diffusion Phabricator c694bd651b49

Make base64 the default SMTP encoding
c694bd651b49
Actions

Description

Make base64 the default SMTP encoding

Summary: Ref T11120. If this works, I'll just remove this option completely.

Test Plan: ¯\_(ツ)_/¯

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T11120

Differential Revision: https://secure.phabricator.com/D16095

Details

Provenance

epriestley	Authored on Jun 9 2016, 8:17 PM
epriestley	Pushed on Jun 9 2016, 8:22 PM

Reviewer

chad

Differential Revision

D16095: Make base64 the default SMTP encoding

Parents

rPe6e4ddd03ab4: Change simple template dashboard to "Open Tasks" instead of "All Tasks"

Branches

Unknown

Tags

Unknown

Tasks

T11120: HTML mails malformed

Build Status

Buildable 12594
Build 15986: Run Core Tests

Event Timeline

epriestley committed rPc694bd651b49: Make base64 the default SMTP encoding (authored by epriestley).Jun 9 2016, 8:22 PM

epriestley added a task: T11120: HTML mails malformed.

Harbormaster completed building B12594: rPc694bd651b49: Make base64 the default SMTP encoding.Jun 9 2016, 8:23 PM

eadler added a subscriber: eadler.Jun 9 2016, 8:41 PM

eadler added inline comments.

/src/applications/config/option/PhabricatorPHPMailerConfigOptions.php
62	base64 here?

Ah, yeah -- I'll fix that if we don't just completely remove the option.

epriestley mentioned this in T12039: Emails from Diffusion contain space or new line character in link making them not clickable.Dec 22 2016, 5:26 PM

Changes (1)

Path

Size

src/

applications/

config/

option/

PhabricatorPHPMailerConfigOptions.php

rPc694bd651b49

View Options

src/applications/config/option/PhabricatorPHPMailerConfigOptions.php