Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Node engine requirement, version 4 support #722

Closed
uiteoi opened this issue Feb 3, 2017 · 5 comments
Closed

Node engine requirement, version 4 support #722

uiteoi opened this issue Feb 3, 2017 · 5 comments

Comments

@uiteoi
Copy link

uiteoi commented Feb 3, 2017

Since version 3.0.0, nodemailer requires node engine ">=6.0.0", while the previous requirement was ">=0.10.0".

This change was made as part of the last commit which title means it is about the licencing change. As a side note, it would have been better to have a separate commit for this change.

Is it possible to at least support node v4, >=4.0.0" which is considered Active in the Long-Term-Support schedule until 2018-04-01?

Reference: LTS schedule

Thanks for this great project.

@andris9
Copy link
Member

andris9 commented Feb 3, 2017

Node v4 was missing some stuff that I wanted to use but is hard to polyfill, like the spread operator. You can keep using v2.7 as long as you want if you need support for older versions of Node. The v3.0.0 was not just about license change, it was a major rewrite, you can see most of the changes from the CHANGELOG.

@andris9 andris9 closed this as completed Feb 3, 2017
@uiteoi
Copy link
Author

uiteoi commented Feb 3, 2017

Thanks for the quick response.

We currently use Node 4.6 in production so we are not be able to upgrade to Nodemailer v3.

Is the spread operator that critical?

Will you continue to support version 2.7 until its end of life of Node 4 - i.e. 2018-04-01?

@andris9
Copy link
Member

andris9 commented Feb 3, 2017

Spread operator is not the only difference but yeah, I want to use it. For example consider these two identical examples:

function abc(){
    var args = Array.prototype.slice.call(arguments);
    var cb = args.pop();
    args.unshift('first');
    cb.apply(null, args);
}

and

function abc(...args){
    var cb = args.pop();
    cb('first', ...args);
}

I'll fix any serious issues with Nodemailer v2 for the foreseeable future. By "serious" I mean something that would cause Nodemailer to throw an error so it would cause the application process to exit or a security issue like RCE.

@uiteoi
Copy link
Author

uiteoi commented Feb 4, 2017

I understand the convenience, but for now I have made the choice to continue to support what Node supports. In a bit more than a year it will be over and we will be able to switch to ES6 for server-only code.

Thanks for the continued support of version 2.7.

@frehner
Copy link

frehner commented Mar 3, 2017

Forgive me if I missed it, but are the docs for v2 gone now that v3 is released?

I'm in the same boat where I'm using nodemailer:2 because of Node:4 dependency (AWS Lambda), and I wasn't able to find any v2 docs.

Or are the v2 and v3 docs similar enough to be mostly interchangeable?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants