[prev in list] [next in list] [prev in thread] [next in thread] 

List:       rt-devel
Subject:    Re: [Rt-devel] Patch review request: Incorrect evaluation of
From:       Dominic Hargreaves <dominic.hargreaves () oucs ! ox ! ac ! uk>
Date:       2010-01-19 10:50:45
Message-ID: 20100119105045.GG3564 () gunboat-diplomat ! oucs ! ox ! ac ! uk
[Download RAW message or body]

[Attachment #2 (multipart/signed)]


On Thu, Jan 14, 2010 at 02:55:59PM +0000, Dominic Hargreaves wrote:
> Hi,
> 
> I've just filed a bug at
> 
> http://rt3.fsck.com/Ticket/Display.html?id=14360
> 
> detailing a recent problem we've discovered following the upgrade
> of one of our RT instances from RT2 to RT 3.8.7.
> 
> I've attached a patch which I've verified fixes the problem on our 
> test system. I would appreciate if someone knowledgable in this area
> of the RT code base could cast their eye over the patch to make sure
> it makes sense before I deploy it on our production system (I am
> slightly nervous since regressions introduced into the mail gateway
> code is likely to be very visible to end users).

After some internal review, I have made a small change to catch a
possible error condition, rolled it out in production, and sent the
patch to the ticket.

-- 
Dominic Hargreaves, Systems Development and Support Team
Computing Services, University of Oxford

["signature.asc" (application/pgp-signature)]

_______________________________________________
List info: http://lists.bestpractical.com/cgi-bin/mailman/listinfo/rt-devel


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic