Skip to content
This repository has been archived by the owner on Aug 11, 2021. It is now read-only.

npm install fails on scoped modules #271

Open
jbeard4 opened this issue Oct 8, 2017 · 0 comments
Open

npm install fails on scoped modules #271

jbeard4 opened this issue Oct 8, 2017 · 0 comments

Comments

@jbeard4
Copy link

jbeard4 commented Oct 8, 2017

Steps to reproduce:

  1. publish a scoped module to npm-registry-couchapp as an admin user
  2. attempt to npm install the scoped module

Expected results: the scoped module should install successfully.
Actual results: the scoped module fails to install with an E404.

I traced this, and the reason is due to the fact that URL to tarballs returned by the show.packages in the design document are not properly escaping the "/" character in the scoped package name.

I believe I have resolved this issue, and will submit a PR. I understand the this project is deprecated and no longer merging PRs, but I wanted to submit this regardless, as it might be useful for someone else.

I would appreciate your feedback. Thank you.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant