ThisIsMissEm / node-smtp-client

[DEPRECATED] An SMTP Client & Server module for Node.js
MIT License
38 stars 6 forks source link

examples use Date().toString() to generate the 'Date' header, which is incorrect #4

Open adamgundy opened 14 years ago

adamgundy commented 14 years ago

the javascript Date().toString() method doesn't create a valid RFC822 'date'. you can see that it's wrong by looking in eg gmail - view the actual message (instead of just the thread in your inbox) and you'll see the time is wrong by several hours (depending on your timezone).

there's an example of creating a valid RFC822 date in javascript here:

http://www.sanctumvoid.net/jsexamples/rfc822datetime/rfc822datetime.html

it would be nice to have a method to do this in the node-smtp library..

ThisIsMissEm commented 14 years ago

Hi Adam,

This library hasn't been maintained since about December / January, I will eventually be rewriting it, so when I do, I'll keep this in mind.

– Micheil

adamgundy commented 14 years ago

is there an alternative library you recommend? I've been using yours (or at least a forked version of it) and it seems to work pretty well.. the only issue I've had is that it doesn't replace newlines with CR/NL (which SMTP expects) so you have to do it manually when building the message..

ThisIsMissEm commented 14 years ago

As far as I know, mine is the only around (along with it's forks), sorry about the lack of work being done on it, although, I did have to wait for a few apis to stabilise and things like that, and then school / work came up so I haven't had much time. (And then there's that cool stuff called WebSockets. :P)