Skip to content

Add support for NPM credentials.

Administrator requested to merge vg/npm-support-credentials into main

Created by: varungandhi-src

The largest part of the change is the testing setup. I'm using Verdaccio to spin up a proxy for the NPM registry, and testing authorization against that.

Questions and TODOs:

  • Should I include any documentation about Verdaccio so that people can change/understand the code better? - Resolution: Removed the usage of Verdaccio altogether.
  • Is there a better/simpler way to test this same behavior? - Resolution: Using a mock server which uses a hard-coded JSON response.
  • I've left a TODO for adding a test case for an error code path I hit, I will fix that.

Merge request reports

Loading