Great Circle Associates Majordomo-Users
(July 1994)
 

Indexed By Date: [Previous] [Next] Indexed By Thread: [Previous] [Next]

Subject: Re: resend header bug
From: Larry Sheldon <lsheldon @ bluejay . creighton . edu>
Date: Fri, 8 Jul 94 19:22:27 CDT
To: awdeacon @ merlot . arc . nasa . gov (Alex Deacon - NAIC)
Cc: rouilj @ cs . umb . edu, colin @ callisto . pas . rochester . edu, majordomo-users @ GreatCircle . COM
In-reply-to: <199407082226.PAA23671@merlot.arc.nasa.gov>; from "Alex Deacon - NAIC" at Jul 08, 94 3:25 pm
Mailer: Elm [revision: 70.85.2.1]

> Collin, (and John)(and Alex)
> 
> > My guess is that it would be in the code that handles the approved
> > header in the body of the message, but I am not sure. I tried running
> > the test case by hand through the 1.92 resend with no luck. If anybody
> > else tracks it down, I would appreciate looking at the patch so I can
> > make sure that it is really fixed and not merely masked in the 1.93
> > resend.
> 
> I *think* I patched a simillar problem I was having a couple of weeks
> ago.  When an owner of a moderated list posted to the list with the
> Approved: command in the BODY of the message, header information (such
> as bulk, sender) was getting mixed in with the text of the message.
> This header info was being inserted after the first (or second, I dont
> remember) line of the message.  And for some reason the subject line
> got blown away in the process also.  
> 
> This behaviour did not occur when the listowner placed the Approved:
> command in the header of the message.  
> 
> I came up with the following patch (for 1.92), its a hack, and hasn't
> been fully tested but it seems to be working well.  Basically the
> header information looked like it was being rewritten when a valid
> Approved line was detected, however if this Approved: line is in the
> body of the message then there is no need to print the header info
> again.
> 
> Comments?
> 
[patch set out]

This sounds like the problem (and the area of code I was looking at) about
a year ago when we first brought up majordomo here--I gave up trying to put
the "Approved" stuff in the body for just the reasons given above.

I was (and am) too poorly trained to attempt very much perl code--and Brent
was up to his ears it appeared, and I had to get something going--we finally
got the approve program and process to run, but it seems like I had to
make some mods to the code that handles "Approved" anywhere run, when
I got the header version running I lost interest in the task.

But I remember thinking that the "Approved-in-the-body" code looked like
a kludge that probably worked at most at one site with some really site-
local conditions.  And as I recall, I decided the reason I was too frightened
to proceed was that the code--having found "Approved:" somewhere, assummed
that it had found it in the headers.

I'm not sure if I have added anything here, but for what ever it is worth,
the description above sounds really familiar.

Oh, and it seems like I determined that the problem is exacerbated if you
"forward" a piece to the list without deleting the MA-inserted blank line
beore the text--the code (it seems like) keyed on the blank line as "eoh",
even if it had already _found_ the end of the headers.

--
-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-
.                                                                       . 
- L. F. (Larry) Sheldon, Jr.                                            -
. Unix Systems Administration                                           . 
- Creighton University Computer Center-Old Gym   Most of us can do more -
. 2500 California Plaza                           than we think we can,  .
- Omaha, Nebraska, U.S.A.  68178                 but usually less than  -
. lsheldon@creighton.edu                          we think we do.       .
- 402 280-2254 (work)                                                   -
. 402 977-2946 (pager)                                  "Bits & Pieces" .
- 402 332-4622 (residence)                                              -
.                                                                       .
.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.-.


References:
Indexed By Date Previous: Re: resend header bug
From: Alex Deacon - NAIC <awdeacon@merlot.arc.nasa.gov>
Next: Re: resend header bug
From: colin sebastian roald <colin@callisto.pas.rochester.edu>
Indexed By Thread Previous: Re: resend header bug
From: Alex Deacon - NAIC <awdeacon@merlot.arc.nasa.gov>
Next: Re: resend header bug
From: colin sebastian roald <colin@callisto.pas.rochester.edu>

Google
 
Search Internet Search www.greatcircle.com