Microsoft's Office365 hosted Exchange servers do not allow users to set the "From" human readable value. It will use the name as set in the Exchange address book.

This is documented by Microsoft at https://support.microsoft.com/en-us/help/4458479/improvements-in-smtp-au....

Proposed resolution:
Update the README.txt file and the SMTP settings form with messages to indicate this incompatibility. I believe this is sufficient.

CommentFileSizeAuthor
#10 3015173-office365-10.patch1.48 KBdiego_mow
#2 3015173-Office365-2.patch1.35 KBpartyka

Comments

partyka created an issue. See original summary.

partyka’s picture

partyka commented
StatusFileSize
new 3015173-Office365-2.patch1.35 KB
partyka’s picture

partyka commented
partyka’s picture

partyka commented
Status: Active » Needs review

It appears that the 8.7 failures are unrelated to this patch.

Status: Needs review » Needs work

The last submitted patch, 2: 3015173-Office365-2.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

chris matthews’s picture

chris matthews commented
Assigned: partyka » Unassigned
Status: Needs work » Needs review
rivimey’s picture

rivimey
she/her
commented

I'm not convinced that this is the right way to include this information. Could it be in documentation, or the Help module?

partyka’s picture

partyka commented

The patch includes a modification to the readme.

My opinion is that since the description setting on the admin form is meant to help the user understand what's going on, it's appropriate there.

That said, I'm not terribly attached to having it there. It should be in the README though.

rivimey’s picture

rivimey
she/her
commented

Hi, ... and I am not strongly opposed as such, just wanted to raise the question. Thanks for your efforts!

diego_mow’s picture

diego_mow commented
StatusFileSize
new 3015173-office365-10.patch1.48 KB

Uploading new patch with a more generic phrase inside Configuration module.

Probably other providers similar to Office365 may have the same effect, so I think it would be useful to make people aware of it.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented 
+++ b/README.txt
@@ -40,3 +40,10 @@ extension is working.  If the SMTP module detects openssl is available it
+Note for Office365 users: ¶
...
+Exchange. Please see this Microsoft KB article for more details: ¶

Apart from miner np on line 46/49 trailing whitespaces which should be removed I set it to RTBC. Nothing harmful and description looks good. +1 But please remove the TW.

dqd’s picture

dqd
Location London | N.Y.C | Paris | Hamburg | Berlin
commented
Status: Needs review » Reviewed & tested by the community
japerry’s picture

japerry
Location KOMK
commented
Status: Reviewed & tested by the community » Fixed

  • japerry committed 32531c3 on 8.x-1.x authored by Diego_Mow 
    Issue #3015173 by partyka, Diego_Mow: Office 365 and Exchange

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.