-
Notifications
You must be signed in to change notification settings - Fork 2.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
feat(js): added a verdaccio listen address to configuration #26976
feat(js): added a verdaccio listen address to configuration #26976
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
☁️ Nx Cloud ReportCI is running/has finished running commands for commit 8e187db. As they complete they will appear below. Click to see the status, the terminal output, and the build insights. 📂 See all runs for this CI Pipeline Execution ✅ Successfully ran 4 targets
Sent with 💌 from NxCloud. |
I could not get documentation to work, maybe one of the maintainers would be so kind to run it for this PR? |
1fc59c1
to
07f53e8
Compare
21e6a41
to
89768f7
Compare
89768f7
to
a0f4ea3
Compare
Added a Verdaccio listen address to the configuration so that you are able to configure the listen address of Verdaccio.
a0f4ea3
to
9f84df3
Compare
Co-authored-by: William van der Vegt <william.van.der.vegt@moba.net> Co-authored-by: Craigory Coppola <craigorycoppola@gmail.com> (cherry picked from commit 50e08aa)
This pull request has already been merged/closed. If you experience issues related to these changes, please open a new issue referencing this pull request. |
Current Behavior
Running Verdaccio through nx always uses localhost by default to listen to.
The Verdaccio config.yaml listen address is always overridden by the executor because it sets the --listen argument.
Expected Behavior
In our situation, where we are developing in a VM we want Verdaccio to listen to all requests by setting it to 0.0.0.0 instead of localhost.
Related Issue(s)
No Issue registered
Fixes #
Added a listen address to the Verdaccio configuration and executor.
Decided to make it a simple implementation to prevent breaking changes.