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

Update axios to latest version #289

Closed
3 of 5 tasks
Tjerk-Haaye-Henricus opened this issue Aug 27, 2023 · 15 comments
Closed
3 of 5 tasks

Update axios to latest version #289

Tjerk-Haaye-Henricus opened this issue Aug 27, 2023 · 15 comments
Labels
bug Something is not working.

Comments

@Tjerk-Haaye-Henricus
Copy link

Preflight checklist

Ory Network Project

No response

Describe the bug

axios version of js client is 0.21.4 newest version is 1.5.0

Reproducing the bug

Install the client look in package json

Relevant log output

No response

Relevant configuration

No response

Version

1.1.51

On which operating system are you observing this issue?

None

In which environment are you deploying?

None

Additional Context

No response

@Tjerk-Haaye-Henricus Tjerk-Haaye-Henricus added the bug Something is not working. label Aug 27, 2023
@thesocialdev
Copy link

What's the status of this issue?

@aeneasr
Copy link
Member

aeneasr commented Oct 5, 2023

We will need to update the generator to address this

@beanow-at-crabnebula
Copy link

Axios 0.x now has a CVE: GHSA-wf5p-g6vw-rhxx
Affecting the TS Ory client.

@aeneasr
Copy link
Member

aeneasr commented Nov 15, 2023

We just release ory/client v1.4.0 which uses the newest generator, hopefully resolving this issue!

@aeneasr aeneasr closed this as completed Nov 15, 2023
@aeneasr
Copy link
Member

aeneasr commented Nov 15, 2023

It looks like the new generator still uses 0.x of Axios :/

"axios": "^0.27.2"

@aeneasr
Copy link
Member

aeneasr commented Nov 15, 2023

Seems like there is work to upgrade axios, but it will take a few weeks before that's being released: OpenAPITools/openapi-generator@a460b7e

@beanow-at-crabnebula
Copy link

Weeks? 😅
I'm not familiar with their release policy, and it looks like they rolled their own version management.
How feasible would it be to run with a commit ref instead of a release?

@beanow-at-crabnebula
Copy link

Coming back to this one, looks like the generator update is released:
https://github.com/OpenAPITools/openapi-generator/releases/tag/v7.2.0

Typescript @ory/client is still affected though:
https://github.com/ory/sdk/blob/master/clients/client/typescript/package.json#L27

https://www.npmjs.com/package/@ory/client/v/1.4.8?activeTab=code

Even though we just updated to the latest generator?
#315

Perhaps need to trigger a build.

@aeneasr
Copy link
Member

aeneasr commented Jan 8, 2024

Perhaps need to trigger a build.

Correct!

@aeneasr aeneasr mentioned this issue Jan 12, 2024
5 tasks
@aeneasr
Copy link
Member

aeneasr commented Jan 12, 2024

Release was triggered yesterday

@Tjerk-Haaye-Henricus
Copy link
Author

Awesome 😎 Thanks a lot

@beanow-at-crabnebula
Copy link

Looking into that, it seems that hasn't resolved the axios version issue though 😢

https://www.npmjs.com/package/@ory/client/v/1.5.1?activeTab=code

"axios": "^0.27.2"

┌─────────────────────┬────────────────────────────────────────────────────────┐
│ moderate            │ Axios Cross-Site Request Forgery Vulnerability         │
├─────────────────────┼────────────────────────────────────────────────────────┤
│ Package             │ axios                                                  │
├─────────────────────┼────────────────────────────────────────────────────────┤
│ Vulnerable versions │ >=0.8.1 <1.6.0                                         │
├─────────────────────┼────────────────────────────────────────────────────────┤
│ Patched versions    │ >=1.6.0                                                │
├─────────────────────┼────────────────────────────────────────────────────────┤
│ Paths               │ [...] > @ory/client@1.5.1 >                            │
│                     │ axios@0.27.2                                           │
├─────────────────────┼────────────────────────────────────────────────────────┤
│ More info           │ https://github.com/advisories/GHSA-wf5p-g6vw-rhxx      │
└─────────────────────┴────────────────────────────────────────────────────────┘
1 vulnerabilities found
Severity: 1 moderate

@BatuhanW
Copy link

I think this issue needs to be re-opened. cc @aeneasr

@aeneasr
Copy link
Member

aeneasr commented Jan 30, 2024

1.5.2 is out with axios in 1.6+

@aeneasr
Copy link
Member

aeneasr commented Jan 30, 2024

https://www.npmjs.com/package/@ory/client/v/1.5.2?activeTab=code

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

No branches or pull requests

5 participants