Merge pull request #58 from Rich-Harris/fix-npmjs-tls-issues

Replace npmjs by yarn registry
Rich-Harris · Sep 1, 2018 · 87b3fe2 · 87b3fe2
2 parents 9b68a00 + 0f776c7
commit 87b3fe2
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/config.js b/config.js
@@ -3,7 +3,10 @@ const sander = require( 'sander' );
 
 exports.root = path.resolve( __dirname );
 exports.tmpdir = process.env.NOW ? `/tmp` : `${exports.root}/.tmp`;
-exports.registry = 'https://registry.npmjs.org';
+
+// TODO(sven): this is a workarround https://twitter.com/svensauleau/status/1035880023964766209
+// exports.registry = 'https://registry.npmjs.org';
+exports.registry = 'https://registry.yarnpkg.com';
 
 if ( !process.env.NOW ) {
 	try {

diff --git a/server/child-processes/create-bundle.js b/server/child-processes/create-bundle.js
@@ -56,7 +56,10 @@ async function createBundle ({ hash, pkg, version, deep, query }) {
 }
 
 function fetchAndExtract ( pkg, version, dir ) {
-	const tarUrl = pkg.versions[ version ].dist.tarball;
+	let tarUrl = pkg.versions[ version ].dist.tarball;
+
+	// TODO(sven): this is a workarround https://twitter.com/svensauleau/status/1035880023964766209
+	tarUrl = tarUrl.replace("npmjs.org", "yarnpkg.com");
 
 	info( `[${pkg.name}] fetching ${tarUrl}` );