Chris Wolfe wrote:
> 
> Found a glitch in org.apache.james.MailHeaders, the isValid tests assume
> that the getHeader return is never null. Since I get a NPE every time I
> omit a header it seems to use a null rather than a zero-length list. I
> would feel much safer using a == null || .length == 0.

Already fixed in the last CVS update. I'll make a release of that next
week.

> 
> Every other mail server I have seen will accept a message without a Date
> header, and I think they will add the header using their current date.
> This seems a more reasonable treatment than rejecting the message
> outright.

These are rfc822 specs (page 17)... 

     message     =  fields *( CRLF *text )       ; Everything after
                                                 ;  first null line
                                                 ;  is message body

     fields      =    dates                      ; Creation time,
                      source                     ;  author id & one
                    1*destination                ;  address required
                     *optional-field             ;  others optional


For what I can understand the rule is ONE Sender, ONE or more recipient
and ONE Date.

Agree that being more coherent with other mail server may be smarter
than being closer to specs... 

+1 for me. Does anybody has better suggestion?

Fede


> 
> Chris
>


------------------------------------------------------------
To subscribe:        [EMAIL PROTECTED]
To unsubscribe:      [EMAIL PROTECTED]
Archives and Other:  <http://java.apache.org/>
Problems?:           [EMAIL PROTECTED]

Reply via email to