search

alexanderGugel / ied

:package: Like npm, but faster - an alternative package manager for Node
http://alexandergugel.github.io/ied
MIT License
1.99k stars 53 forks source link

Fix broken install for package using caret version #165

Closed stephanebachelier closed 8 years ago

stephanebachelier commented 8 years ago
Checklist

none

Description of change

Fix #164

Needle package has released a new version (1.1.0) with automatically encode URI which result in a 404 for any call to NPM registry with a version range.

Example: babel-plugin-transform-es2015-parameters >=6.9.0 <7.0.0 https://registry.npmjs.org/babel-plugin-transform-es2015-parameters/%3E=6.9.0%20%3C7.0.0

If you try this URL in a browser or with CURL it will works and give you a valid response with a 200 status code but not with needle.

stephanebachelier commented 8 years ago

I can confirm that the encodeURI is the problem.

I've made a PR to needle (https://github.com/tomas/needle/pull/185) to add encode_uri option and test ied with my branch and it works as expected and tests are green.

alexanderGugel commented 8 years ago

Great PR! Thanks!

alexanderGugel commented 8 years ago

published as 2.3.2