Open joserodriguezjll opened 11 months ago
this is an important fix needed, is there any reason the tests are not running ?
this is an important fix needed, is there any reason the tests are not running ?
No idea, let's see if some of the product developers can check this issue and check the problem with the tests
this is an important fix needed, is there any reason the tests are not running ?
No idea, let's see if some of the product developers can check this issue and check the problem with the tests
Hi there, I approved the tests run, this change is failing at build validation. I have added this item to the backlog to be prioritised soon.
can someone look at this error, doesn't look like an axios problem but with the test itself Thanks
Hi! I will be looking into this and include it in the next release
Hi @tiwarishubham635, when will the next release be?
Hi @tiwarishubham635, when will the next release be?
This thursday
Hi @tiwarishubham635, when will the next release be?
This thursday
Great, thanks!
@tiwarishubham635 will other packages that use the @sendgrid/client
(like @sendgrid/mail
) be updated as well?
@tiwarishubham635 will other packages that use the
@sendgrid/client
(like@sendgrid/mail
) be updated as well?
Yes, since the changes are in the package version of axios, all the affected places will be modified. I hope that answers your query.
@tiwarishubham635 The current version of Axios only supports Node.js 12.x and above: https://github.com/axios/axios/blob/9588fcdec8aca45c3ba2f7968988a5d03f23168c/.github/workflows/ci.yml#L15
Node.js 6 is no longer in Long Term Support and has security vulnerabilities. My recommendation would be to treat this as a breaking change, drop support for older unsupported Node.js versions, then bump the major version.
You'll likely find that the these tests for Node.js 7, 8, and 10 will also fail the same way - they were skipped because the test for 6 failed.
@tiwarishubham635 The current version of Axios only supports Node.js 12.x and above: https://github.com/axios/axios/blob/9588fcdec8aca45c3ba2f7968988a5d03f23168c/.github/workflows/ci.yml#L15
Node.js 6 is no longer in Long Term Support and has security vulnerabilities. My recommendation would be to treat this as a breaking change, drop support for older unsupported Node.js versions, then bump the major version.
You'll likely find that the these tests for Node.js 7, 8, and 10 will also fail the same way - they were skipped because the test for 6 failed.
Yes, we have already identified this issue and are in the process of removing the older node versions. Once it is done, this PR will be merged
@tiwarishubham635 @shrutiburman "Thursday" is Not sure if it was last Thursday or this Thursday, but it looks like they did the old Node.js drop in #1390, so hopefully it will be merged soon.
Hi There, Yes, the PR is in review, once it's merged the changes will be picked up in the upcoming release next week. We follow a bi-weekly release cadence.
@joserodriguezjll when you have a chance can you merge main
into your branch?
Okay, guys, maybe it will get updated in a few weeks.
What is the ETA to get this merged & released?
@joserodriguezjll when you have a chance can you merge
main
into your branch?
This is a pretty critical issue for us and many others - perhaps the sendgrid team might want to drive this forwards without waiting for an external contributor?
Na guys, main question is when will the npm package will get updated? I think the already merged/fixed the issue in Github repo, but not updated the npm module, btw is this correct npm module na? Because I am using this in my projects.
Let's gooooooooooooooooooooooooooooooooo.
Has anyone had success using 8.0.0? I am seeing an issue as described here https://github.com/sendgrid/sendgrid-nodejs/pull/1391#issuecomment-1846451872 - axios3 is not a function
Also is there a reason Issues are disabled on this GitHub repo? I would like to raise this in the right place rather than in the comments section on several PRs.
Has anyone had success using 8.0.0? I am seeing an issue as described here #1391 (comment) -
axios3 is not a function
Also is there a reason Issues are disabled on this GitHub repo? I would like to raise this in the right place rather than in the comments section on several PRs.
Using it since the launch, never faced any issue till now mate.
Has anyone had success using 8.0.0? I am seeing an issue as described here #1391 (comment) -
axios3 is not a function
Also is there a reason Issues are disabled on this GitHub repo? I would like to raise this in the right place rather than in the comments section on several PRs.
I was thinking to shift from sendgrid soon, their service getting poor, & new clients onboarding issue too.
This seems to be addressed in #1394, but three months to fix a vulnerability flagged as critical? Yikes.
Fixes
Checklist
If you have questions, please file a support ticket.