Bug 458026 - In-Reply-To in notification emails changed format; threading broke
Summary: In-Reply-To in notification emails changed format; threading broke
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Bugzilla
Classification: Community
Component: Bugzilla General
Version: devel
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: David Lawrence
QA Contact:
URL:
Whiteboard:
Depends On:
Blocks: 1318162
TreeView+ depends on / blocked
 
Reported: 2008-08-06 07:00 UTC by Mark McLoughlin
Modified: 2016-03-16 08:10 UTC (History)
0 users

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Clone Of:
: 1318162 (view as bug list)
Environment:
Last Closed: 2008-08-06 16:31:39 UTC
Embargoed:


Attachments (Terms of Use)
Patch to fix domain name in the in-reply-to mail header (v1) (983 bytes, patch)
2008-08-06 15:38 UTC, David Lawrence
nelhawar: review+
Details | Diff

Description Mark McLoughlin 2008-08-06 07:00:25 UTC
Not worth fixing unless easy to fix, but the email headers used to contain:

  In-Reply-To: <bug-446214-196610.com>

but now contain

  In-Reply-To: <bug-446214-196610.redhat.com/>

so new email threads have been created.

It'd be nice to go back to the old format again

Comment 1 David Lawrence 2008-08-06 15:04:40 UTC
Verified this is indeed broken. Taking and will fix in the next update.

Comment 2 David Lawrence 2008-08-06 15:38:37 UTC
Created attachment 313587 [details]
Patch to fix domain name in the in-reply-to mail header (v1)

Patch attached for review to fix the domain name that is in reply to header.
In upstream bugzilla they add the http(s) to the domain name for reasons I am not sure. In 2.18 we had bugzilla.redhat.com as the domain name hard coded into the email template.

The patch hacks the BugMail.pm code to remove the https from the domain name and the trailing slash so it better matches 2.18 behaviour.

Please review

Dave

Comment 3 Noura El hawary 2008-08-06 16:22:52 UTC
Comment on attachment 313587 [details]
Patch to fix domain name in the in-reply-to mail header (v1)

looks Good Dave, I saw it works fine on bz-web2.

Noura

Comment 4 David Lawrence 2008-08-06 16:31:39 UTC
Thanks Noura. Checking into CVS and will be in the next update.


Note You need to log in before you can comment on or make changes to this bug.