134050 - the return receipts we send out don't have our user agent

Status: VERIFIED INVALID

(MailNews Core :: Backend, defect, P2)

Description

[(not reading, please use [e-mail] instead)]

the return receipts we send out don't have our user agent.

my guess is that they should.

Comment 1

[Jeff Tsai]

Yes, we do. We package it into the "Reporting-UA:" header based on the spec. 
Below are sections of code from nsMsgMdnGenerator.cpp.

...
    nsCOMPtr<nsIHttpProtocolHandler> pHTTPHandler = 
        do_GetService(NS_NETWORK_PROTOCOL_CONTRACTID_PREFIX "http", &rv); 
    if (NS_SUCCEEDED(rv) && pHTTPHandler)
    {
        nsCAutoString userAgentString;
        pHTTPHandler->GetUserAgent(userAgentString);

        if (!userAgentString.IsEmpty())
        {
            tmpBuffer = PR_smprintf("Reporting-UA: %s" CRLF,
                                    userAgentString.get());
            PUSH_N_FREE_STRING(tmpBuffer);
        }
    }
...

Comment 2

[grylchan]

this what I see in my return receipt using Seth's windows build:
Reporting-UA: Mozilla/5.0 (Windows; U; WinNT4.0; en-US; rv:0.9.9+) 
Gecko/20020325

It's the first line in 2nd part (Mdn part2.txt)of return receipt..

Comment 3

[Jeff Tsai]

Seth, I am marking this as invalid since we do have the user agent information 
added to the Reporting-UA: header instead of X-Mailer: header.

Comment 4

[[e-mail] (gone - use [e-mail] instead)]

Verified - we have Reporting-UA: Mozilla/5.0 (Windows; U; Windows NT 5.0; en-US;
rv:1.2a) Gecko/20020910