-
Notifications
You must be signed in to change notification settings - Fork 386
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
medplum aws update-app
hard coded to use public npm registry
#4655
Comments
Upon further investigation, it seems as though simply replacing As an alternative approach, perhaps it makes more sense to allow users a way to provide the path to the extracted tar for the https://github.com/medplum/medplum/blob/main/packages/cli/src/aws/update-app.ts#L40 |
Hi @dbryant4-cj - thanks for filing, this is a good issue. |
Background
In our environment, access to the public Internet is very restrictive, meaning access to the public npm registry is not allowed. Instead, we have a Nexus artifact server that serves as the npm registry. This is a requirement of our security posture that we cannot easily change.
Issue
It appears as though the public npm registry is hard coded in the file below which causes the
medplum aws update-app
to fail. Can this be changed to be configurable, perhaps by an environment variable or a CLI option?medplum/packages/cli/src/aws/update-app.ts
Lines 69 to 73 in fe7eb44
The text was updated successfully, but these errors were encountered: