-
Notifications
You must be signed in to change notification settings - Fork 171
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
Moving from request to axios #348
Comments
Are there any features that axios gives us that are currently not implemented with |
Per se no, but why using a client with many and old dependencies, including deprecated ones, when other more updated clients can do the same?
|
That makes sense to me. We recently switched to I've used Feel free to give it a shot, just make sure to test it heavily with a real Jira API. I'm not sure our unit tests are sufficient proof for a change like this. |
I will test against my company JIRA and also using a brand-new cloud JIRA instance which I just provisioned. |
It might be a good idea to setup a free jira organization and create some integration tests against it |
Hi, We are also using Jira Client which uses request module which further uses an old version of tough-cookie module (which has security vulnderabilities). |
Current HTTP client is old, any appetite @pioug if I help moving to axios maybe?
The text was updated successfully, but these errors were encountered: