INN 2.3.2: nnrpd rejects article with duplicate "Cc" header but shouldn't

[greg andruk]

On Sun, Sep 23, 2001 at 11:02:13PM -0400, Jonathan Kamens wrote:
> If you try to post an article through nnrpd in INN 2.3.2 with two "Cc"
> headers, the server rejects it.

That's a limitation of the traditional nnrpd header parser.  Known headers
are limited to three states: HTreq (allow exactly one instance), HTstd
(allow zero or one instance) and HTobs (don't allow this header).

nnrpd treats To, Cc and Bcc as known headers because it offers the ability
to strip them.

The fix is to remove those three headers from the table at the top of
post.c, also remove the lines (also in post.c) that clear out
HDR__CC/HDR__TO/HDR__BCC.

If stripping is still desired, one can either use an embedded filter to do
any desired stripping or use string compares rather than the table to decide
if they belong in OtherHeaders[].  Marginally slower, but nnrpd doesn't need
to squeeze out every last cycle the way innd does.