Menu

#8 Error for mails in sent mail folder

open
nobody
3
2007-10-03
2007-10-02
No

I finally found the time to update to 1.2.4, but with one Stamp I get the following error:

TypeError: localDt has no properties

This is displayed in the stamp info dialog. The Error Console gives me further details:

Error: localDt has no properties
Source File: chrome://ppost/content/messageWindowOverlay.js
Line: 292

The tooltip says "The stamp could not be verified".

I had a look at the code and guess that's because a mail in my sent mail folder hasn't been through any SMTP server and thus hasn't any Received headers.

I guess that's my own fault, as I suggested relying on the Received headers in the first place. I'm not sure what the suggested behaviour would be. Maybe you should use the Date as a fallback when there are no Received headers.

I'm still of the opinion that all these expiry worries should be solved if you implement a duplicate stamp database. So this is only to inform you that the current code does something you probably did not intend.

Discussion

  • Martin von Gagern

    • priority: 5 --> 3
     
  • Ali Lokhandwala

    Ali Lokhandwala - 2007-10-04

    Logged In: YES
    user_id=1476495
    Originator: NO

    Hi Martin,
    I know of this behaviour and will have a proper message in place instead of the raw log error.
    Usually sent mail will be stamped for someone else (i.e. not your own address), so anyways it will appear un-stamped for you. Its just because we are testing by sending mail to ourselves that this happens.

    -Ali

     
  • Martin von Gagern

    Logged In: YES
    user_id=733620
    Originator: YES

    There is a real life scenario for this as well: if you sent mail to a mailing list which you have subscribed yourself as well. Then you would want to add that list address to your pennypost config.

     
  • Ali Lokhandwala

    Ali Lokhandwala - 2007-10-14

    Logged In: YES
    user_id=1476495
    Originator: NO

    Error better handled in v1.3 .. but still will need working on I agree!

    -Ali

     

Log in to post a comment.

MongoDB Logo MongoDB