Bug 1477476

Summary: libreport-plugin-mailx don't respect the EmailFrom/EmailTo settings when write it in '"'
Product: Red Hat Enterprise Linux 7 Reporter: Frank Büttner <bugzilla>
Component: libreportAssignee: abrt <abrt-devel-list>
Status: CLOSED WONTFIX QA Contact: BaseOS QE - Apps <qe-baseos-apps>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 7.3CC: djez, mkutlak
Target Milestone: rc   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2018-06-15 08:02:17 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Frank Büttner 2017-08-02 08:01:36 UTC 
Description of problem:
When using it to send an e-mail the header of the mail contains an incorrect "From"/"To" field.

But the from/to part at the smtp transfer part are ok.
The error happens after the data command.


Version-Release number of selected component (if applicable):
libreport-plugin-mailx-2.1.11-35

How reproducible:
Every time

Steps to Reproduce:
1. let an application crash


Actual results:
The generated email will contain in invalid "From"/"To" field in the header.
From: <"foo"@localhost.localdomain>
To: <"bar" @localhost.localdomain>

But the the smtp part before the data command:
MAIL FROM:<foo> SIZE=15023
RCPT TO:<bar> ORCPT=rfc822;bar
DATA


Expected results:
An correct mail header.


Additional info:
/etc/libreport/plugins/mailx.conf:
EmailFrom="foo"
EmailTo="bar"

using:
EmailFrom=foo
EmailTo=bar
will work. But on http://abrt.readthedocs.io/en/latest/examples.html the fields are written with '"'.
Comment 2 Martin Kutlak 2018-06-15 08:02:17 UTC 
Thank you for the report.

The example in the documentation is actually wrong because we don't support quoted values in configuration files.

I've corrected the example and added a line with information explaining what is expected in the configuration file.

Updated docs:
https://abrt.readthedocs.io/en/latest/conf.html#configuration-files
https://abrt.readthedocs.io/en/latest/examples.html#examples

PR changing the docs:
https://github.com/abrt/doc/pull/15