Skip to content
New issue

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

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

Already on GitHub? # to your account

Allow any registry to be used for auditing #6484

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cheesestringer
Copy link

Summary

Allow auditing against any registry. This is useful for auditing against a private or whitelisted registry.
For example, in our organisation npmjs is allowed while yarnpkg is blocked :(

Test plan

Command: yarn audit
Output: An unexpected error occurred: "https://registry.yarnpkg.com/-/npm/v1/security/audits: getaddrinfo ENOTFOUND registry.yarnpkg.com registry.yarnpkg.com:443".

Command: yarn audit --registry https://registry.npmjs.org
Output: 0 vulnerabilities found - Packages audited: 46500

Command: yarn audit --registry https://artifactory.dev.company/artifactory/api/npm/npm-dev
Output: error Your configured registry "https://artifactory.dev.company/artifactory/api/npm/npm-dev" does not support audit requests.

@adamscybot
Copy link

Also experiencing this same problem

@eugenesimakin
Copy link

Duplicate of #7263.

Hi @cheesestringer, I suggest you closing this pull request and applying your knowledge on reviewing the #7263

Fix linting error
Fix linting error
@cheesestringer
Copy link
Author

Forgot all about this. Fixed the conflicts but happy to close this for the newer PR.

@HansAarneLiblik
Copy link

Any updates regarding this?

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

Successfully merging this pull request may close these issues.

4 participants