From the mailinglist (archive see URL): 
 
On Tuesday 17 August 2004 04:29 CET Theo Van Dinter wrote: 
> On Tue, Aug 17, 2004 at 12:33:28AM +0200, Malte S. Stretz wrote: 
> > While I tracked another possible bug (see [1]), I had a look how 
> > check_dcc works.  And was completely puzzled by the code. 
> 
> Heh. 
> 
> > check_dcc() calls 
> >   $self->get('X-DCC-(?:[^:]+-)?Metrics') 
> > to shortcut some ocde path.  That's obviously intended to be a RE.  But 
> > on the whole path through the code I can't find the point where the 
> > header is matched via a RE. 
> 
> Well, now that you mention it...  PMS::get() doesn't accept an RE. 
> 
> > Follow me through the code, maybe you can enlighten me: 
> > 2. PerMsgStatus.pm:1175: get('X-DCC-(?:[^:]+-)?Metrics', undef) 
> 
> Yeah, that's not going to work. 
> 
> > -> PerMsgStatus.pm:1227: calls Message::Node::get_header() 
> 
> doesn't do an RE either. 
> 
> > Ok, so where's the RE applied?  Even if other code paths were taken I 
> > can't find any point where the string is made an RE and I can't believe 
> > that the mail parser puts the "Header" check_dcc() wants into the 
> > cache-hash. 
> 
> Yeah, so we ought to replace: 
> 
>   $_ = $self->get('X-DCC-(?:[^:]+-)?Metrics'); 
>   return 1 if /bulk/; 
> 
> with: 
> 
>   return 1 if grep(/^X-DCC-(?:[^:]+-)?Metrics:/ && /bulk/, 
> @{$self->{msg}->get_all_headers()}); 
 
On Tuesday 17 August 2004 16:04 CET Theo Van Dinter wrote: 
> On Mon, Aug 16, 2004 at 08:56:58PM -0700, Dan Quinlan wrote: 
> > >   return 1 if grep(/^X-DCC-(?:[^:]+-)?Metrics:/ && /bulk/, 
> > > @{$self->{msg}->= get_all_headers()}); 
> > 
> > If we can expand out the list of headers, we should just go with that. 
> > Fetching the entire list of headers is definitely inefficient unless we 
> > really need it. 
> 
> Sure.  I was reproducing the initial behavior, but if we know the list 
> of headers, that'd be better.  The issue, however, is: 
> 
> [...] 
>      DCC SMTP header lines are of the form: 
> 
>      X-DCC-brand-Metrics: chost server-ID; bulk cknm1=count cknm2=count 
> ... where 
>         brand   is the "brand name" of the DCC server, such as 
> "RHYOLITE". [...] 
> 
> So "brand" is open to how things are configured and which DCC server is 
> being used.  I think we have to do the get_all_headers() call there.