-
Notifications
You must be signed in to change notification settings - Fork 3.3k
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
[QUESTION] registry.npmjs.org API/format change for security advisories #1291
Comments
Other changes to the NPM Advisory API include modifications to the 'next' and 'prev' fields which prevent the API from being paginated without having to create workarounds. |
cc @evilpacket |
@ljharb thanks for the ping. On the one hand this API was undocumented and is only supported through the official channel that is npm cli (for npm audit) and the website. On the other this is an unexpected change and I've been informed that engineering will likely be working on a fix for it. Triage pending. |
I just took a look at the advisories output from the referenced endpoint and I think dates should be back to what you are expecting?
Again I want to note that these endpoints are undocumented and unsupported and are likely to change in the future. You may want to consider migrating to consume advisory data from the GitHub Advisory database which is available under the Creative Commons Attribution 4.0 license |
What / Why
Apparently the endoint at https://registry.npmjs.org/-/npm/v1/security/advisories changed the output format without notice.
Previously, timestamps included a "Z" as a timezone indicator - which was now silently removed, breaking e.g. the DependencyTrack parser.
Is the current version the "new stable" or will it be changed again?
Where
Who
References
The text was updated successfully, but these errors were encountered: