Merge branch 'master' into alicanc-run-tests

__tests__/__mocks__/request.js

@@ -49,11 +49,13 @@ const httpMock = {
   request(options: Object, callback?: ?Function): ClientRequest {
     const alias = getRequestAlias(options);
     const loc = path.join(CACHE_DIR, `${alias}.bin`);
+    // allow the client to bypass the local fs fixture cache by adding nocache to the query string
+    const allowCache = options.uri.href.indexOf('nocache') == -1;
 
     // TODO better way to do this
-    const httpModule = options.port === 443 ? https : http;
+    const httpModule = options.uri.href.startsWith('https:') ? https : http;
 
-    if (fs.existsSync(loc)) {
+    if (allowCache && fs.existsSync(loc)) {
       // cached
       options.agent = null;
       options.socketPath = null;

__tests__/fixtures/certificates/cacerts.pem

@@ -0,0 +1,57 @@
+the first CA is a random CA not used in this test, to verify
+that multiple CAs work
+-----BEGIN CERTIFICATE-----
+MIIFjTCCA3WgAwIBAgIRANOxciY0IzLc9AUoUSrsnGowDQYJKoZIhvcNAQELBQAw
+TzELMAkGA1UEBhMCVVMxKTAnBgNVBAoTIEludGVybmV0IFNlY3VyaXR5IFJlc2Vh
+cmNoIEdyb3VwMRUwEwYDVQQDEwxJU1JHIFJvb3QgWDEwHhcNMTYxMDA2MTU0MzU1
+WhcNMjExMDA2MTU0MzU1WjBKMQswCQYDVQQGEwJVUzEWMBQGA1UEChMNTGV0J3Mg
+RW5jcnlwdDEjMCEGA1UEAxMaTGV0J3MgRW5jcnlwdCBBdXRob3JpdHkgWDMwggEi
+MA0GCSqGSIb3DQEBAQUAA4IBDwAwggEKAoIBAQCc0wzwWuUuR7dyXTeDs2hjMOrX
+NSYZJeG9vjXxcJIvt7hLQQWrqZ41CFjssSrEaIcLo+N15Obzp2JxunmBYB/XkZqf
+89B4Z3HIaQ6Vkc/+5pnpYDxIzH7KTXcSJJ1HG1rrueweNwAcnKx7pwXqzkrrvUHl
+Npi5y/1tPJZo3yMqQpAMhnRnyH+lmrhSYRQTP2XpgofL2/oOVvaGifOFP5eGr7Dc
+Gu9rDZUWfcQroGWymQQ2dYBrrErzG5BJeC+ilk8qICUpBMZ0wNAxzY8xOJUWuqgz
+uEPxsR/DMH+ieTETPS02+OP88jNquTkxxa/EjQ0dZBYzqvqEKbbUC8DYfcOTAgMB
+AAGjggFnMIIBYzAOBgNVHQ8BAf8EBAMCAYYwEgYDVR0TAQH/BAgwBgEB/wIBADBU
+BgNVHSAETTBLMAgGBmeBDAECATA/BgsrBgEEAYLfEwEBATAwMC4GCCsGAQUFBwIB
+FiJodHRwOi8vY3BzLnJvb3QteDEubGV0c2VuY3J5cHQub3JnMB0GA1UdDgQWBBSo
+SmpjBH3duubRObemRWXv86jsoTAzBgNVHR8ELDAqMCigJqAkhiJodHRwOi8vY3Js
+LnJvb3QteDEubGV0c2VuY3J5cHQub3JnMHIGCCsGAQUFBwEBBGYwZDAwBggrBgEF
+BQcwAYYkaHR0cDovL29jc3Aucm9vdC14MS5sZXRzZW5jcnlwdC5vcmcvMDAGCCsG
+AQUFBzAChiRodHRwOi8vY2VydC5yb290LXgxLmxldHNlbmNyeXB0Lm9yZy8wHwYD
+VR0jBBgwFoAUebRZ5nu25eQBc4AIiMgaWPbpm24wDQYJKoZIhvcNAQELBQADggIB
+ABnPdSA0LTqmRf/Q1eaM2jLonG4bQdEnqOJQ8nCqxOeTRrToEKtwT++36gTSlBGx
+A/5dut82jJQ2jxN8RI8L9QFXrWi4xXnA2EqA10yjHiR6H9cj6MFiOnb5In1eWsRM
+UM2v3e9tNsCAgBukPHAg1lQh07rvFKm/Bz9BCjaxorALINUfZ9DD64j2igLIxle2
+DPxW8dI/F2loHMjXZjqG8RkqZUdoxtID5+90FgsGIfkMpqgRS05f4zPbCEHqCXl1
+eO5HyELTgcVlLXXQDgAWnRzut1hFJeczY1tjQQno6f6s+nMydLN26WuU4s3UYvOu
+OsUxRlJu7TSRHqDC3lSE5XggVkzdaPkuKGQbGpny+01/47hfXXNB7HntWNZ6N2Vw
+p7G6OfY+YQrZwIaQmhrIqJZuigsrbe3W+gdn5ykE9+Ky0VgVUsfxo52mwFYs1JKY
+2PGDuWx8M6DlS6qQkvHaRUo0FMd8TsSlbF0/v965qGFKhSDeQoMpYnwcmQilRh/0
+ayLThlHLN81gSkJjVrPI0Y8xCVPB4twb1PFUd2fPM3sA1tJ83sZ5v8vgFv2yofKR
+PB0t6JzUA81mSqM3kxl5e+IZwhYAyO0OTg3/fs8HqGTNKd9BqoUwSRBzp06JMg5b
+rUCGwbCUDI0mxadJ3Bz4WxR6fyNpBK2yAinWEsikxqEt
+-----END CERTIFICATE-----
+comments should be stripped
+-----BEGIN CERTIFICATE-----
+MIIDfzCCAmegAwIBAgIJAM4nWEf/MeHVMA0GCSqGSIb3DQEBCwUAMFYxCzAJBgNV
+BAYTAlhYMRUwEwYDVQQHDAxEZWZhdWx0IENpdHkxHDAaBgNVBAoME0RlZmF1bHQg
+Q29tcGFueSBMdGQxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xNjEwMTIwNTQ3NDla
+Fw0yNjEwMTAwNTQ3NDlaMFYxCzAJBgNVBAYTAlhYMRUwEwYDVQQHDAxEZWZhdWx0
+IENpdHkxHDAaBgNVBAoME0RlZmF1bHQgQ29tcGFueSBMdGQxEjAQBgNVBAMMCWxv
+Y2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALVCI5Ma6AR0
+oDv/OqactDMR4pA6CZJnNDYbjRIkjsXi3pXuAXbQ8J/lD7EKNhu8wxrM0PZvdE1s
+ERjgFmZlYYTJkoDQUr9HagwAhAHrUuAq6FsmogLOl1L4QmeddCxExLdTePJAvVxc
++fr3mk7I9Kt5FV1kDOZMyGqkLwKvcGXjx4ue+ZMgQZjWoU4Om7ktA57siAUkrxQg
+SonEnlSAlEQy6/wRSR2XQ85e+o1JHMIti4+h/Soo4BmHetec7zKvHjsL1kuT9s43
+YiVlj660cc/Rmqt61OqPrumeKeVLRLmOD71l0yjU8QEwvAvvzFwdTZQSPUVGb4Je
+9Apxn7MNnKMCAwEAAaNQME4wHQYDVR0OBBYEFFzPIkvacjl8MFC6eRCwSTyW5z9c
+MB8GA1UdIwQYMBaAFFzPIkvacjl8MFC6eRCwSTyW5z9cMAwGA1UdEwQFMAMBAf8w
+DQYJKoZIhvcNAQELBQADggEBAKN9RXWgMuEwhLYzG///duWTcIC7UfamGDVlezxa
+BkR+VGkRSlo4v+MaZKscG2D/NGh4PP5PQZr7okLQY6MIKmcFkIN1BDziEVfKICFs
+AIrcajDLyHaLcAmZv2VJe1sz12pmGZG7uTOncMAngZoNDNI0f4djzvmRd9nn4yGo
+o8vhLMzgdXhp3T7yaCjpbpZUd1bnggJXz9MO76EBcCkS3+HcRRr/0KMD/tk5tZUx
+s35hnRrJi9HvhFjZbJA/KGG8DJp4oyzEfbufmUJ7OdbMn++W3NtG1BrRhbKmH5og
+tpfAr88iJ6BFaXV/4JIxc4Fga8dvjJR3ueh5pT2om/rUzkQ=
+-----END CERTIFICATE-----
+another comment

__tests__/fixtures/certificates/server-cert.pem

@@ -0,0 +1,21 @@
+-----BEGIN CERTIFICATE-----
+MIIDfzCCAmegAwIBAgIJAM4nWEf/MeHVMA0GCSqGSIb3DQEBCwUAMFYxCzAJBgNV
+BAYTAlhYMRUwEwYDVQQHDAxEZWZhdWx0IENpdHkxHDAaBgNVBAoME0RlZmF1bHQg
+Q29tcGFueSBMdGQxEjAQBgNVBAMMCWxvY2FsaG9zdDAeFw0xNjEwMTIwNTQ3NDla
+Fw0yNjEwMTAwNTQ3NDlaMFYxCzAJBgNVBAYTAlhYMRUwEwYDVQQHDAxEZWZhdWx0
+IENpdHkxHDAaBgNVBAoME0RlZmF1bHQgQ29tcGFueSBMdGQxEjAQBgNVBAMMCWxv
+Y2FsaG9zdDCCASIwDQYJKoZIhvcNAQEBBQADggEPADCCAQoCggEBALVCI5Ma6AR0
+oDv/OqactDMR4pA6CZJnNDYbjRIkjsXi3pXuAXbQ8J/lD7EKNhu8wxrM0PZvdE1s
+ERjgFmZlYYTJkoDQUr9HagwAhAHrUuAq6FsmogLOl1L4QmeddCxExLdTePJAvVxc
++fr3mk7I9Kt5FV1kDOZMyGqkLwKvcGXjx4ue+ZMgQZjWoU4Om7ktA57siAUkrxQg
+SonEnlSAlEQy6/wRSR2XQ85e+o1JHMIti4+h/Soo4BmHetec7zKvHjsL1kuT9s43
+YiVlj660cc/Rmqt61OqPrumeKeVLRLmOD71l0yjU8QEwvAvvzFwdTZQSPUVGb4Je
+9Apxn7MNnKMCAwEAAaNQME4wHQYDVR0OBBYEFFzPIkvacjl8MFC6eRCwSTyW5z9c
+MB8GA1UdIwQYMBaAFFzPIkvacjl8MFC6eRCwSTyW5z9cMAwGA1UdEwQFMAMBAf8w
+DQYJKoZIhvcNAQELBQADggEBAKN9RXWgMuEwhLYzG///duWTcIC7UfamGDVlezxa
+BkR+VGkRSlo4v+MaZKscG2D/NGh4PP5PQZr7okLQY6MIKmcFkIN1BDziEVfKICFs
+AIrcajDLyHaLcAmZv2VJe1sz12pmGZG7uTOncMAngZoNDNI0f4djzvmRd9nn4yGo
+o8vhLMzgdXhp3T7yaCjpbpZUd1bnggJXz9MO76EBcCkS3+HcRRr/0KMD/tk5tZUx
+s35hnRrJi9HvhFjZbJA/KGG8DJp4oyzEfbufmUJ7OdbMn++W3NtG1BrRhbKmH5og
+tpfAr88iJ6BFaXV/4JIxc4Fga8dvjJR3ueh5pT2om/rUzkQ=
+-----END CERTIFICATE-----

__tests__/fixtures/certificates/server-key.pem

@@ -0,0 +1,28 @@
+-----BEGIN PRIVATE KEY-----
+MIIEvgIBADANBgkqhkiG9w0BAQEFAASCBKgwggSkAgEAAoIBAQC1QiOTGugEdKA7
+/zqmnLQzEeKQOgmSZzQ2G40SJI7F4t6V7gF20PCf5Q+xCjYbvMMazND2b3RNbBEY
+4BZmZWGEyZKA0FK/R2oMAIQB61LgKuhbJqICzpdS+EJnnXQsRMS3U3jyQL1cXPn6
+95pOyPSreRVdZAzmTMhqpC8Cr3Bl48eLnvmTIEGY1qFODpu5LQOe7IgFJK8UIEqJ
+xJ5UgJREMuv8EUkdl0POXvqNSRzCLYuPof0qKOAZh3rXnO8yrx47C9ZLk/bON2Il
+ZY+utHHP0ZqretTqj67pninlS0S5jg+9ZdMo1PEBMLwL78xcHU2UEj1FRm+CXvQK
+cZ+zDZyjAgMBAAECggEBAKRSKlgRG2f2ptDdaDlldMOboi6oPsc3wpCO14wsEjb5
+nlqDo1YowwvhqCESpcztil7AcWwHzILnxnQrqoL3w7mS17rpoSqBPnVU/leTE9Xf
+cDg6RMOQsITqRaETkB8V1NRx2wKbiE+0hndrgruL2KufIKxCqKMb1tE+uNORYq8q
+kFIBvUnhjLFzepZ749wYTx2Tkdrfe3Y3sW2a17LRr43s0Z41skX/9iz7GdIW8OLD
+Edfw7COvlxZQjtmLqwmoFd5YjOo8u6C6kr74H8OLplGN35cPc+L57kFqKvxV42Lq
+y+EirIZ3b8uItsuOPIf92snpfK9CmdwTCx5lrMmpqjkCgYEA8QKr8xmI3fhUxPD8
+9PYQlIa+pKMtX1sKgS3sA2ZD9bwRC9HmkXhYTYsrakbX2VaGe1dmc/7KMaNaVkrT
++BfMk9SACnoul7dXzV6amzM7AdrZYkJgejbmSJtYGz3YnHQTz4ipaffHfsKoWdvs
+LdvPgBBdfbYsvGxB5ya9kvcsLyUCgYEAwIgawo1mU5KC8BXOvzKSKJ6vTULtnF6d
+zf1OGv6atrS36Sg3ycqPL+kUy44bl1ckjh+VMp6rty/5Z/K1PKkyNniznxj9Epwu
+4O/aI/Q7SrKZmjH64gXWBmyj8doNHM3nFF9mSIjRromovuUeOcpFlATBCe0Pxpxt
+sGlhjnrMVicCgYBvIwlBv9uiaBpG+s3a9AEvTHdrGigZGbVdXlzAMI9UKNY/ehp1
+qGYn0+5AQszUVxcKl4ISKUL54tcMhdL7S5Y18T7eFfuYUJ53gJGQ0e366/1kVzGA
+CgLlJmVZoopZkxlzkRR2XiErbf4N+eEOQJeN+X3zM2ert8woGHBA7iP81QKBgBi0
+3oo8zvbGhFr+0WsjuDHSOzi07/zy/1khulYoef4cLsWSzaXtgnZpeKuubsf6/Mvo
+LaMzTWHSnDTEppFEPRdUYeh2snMi67kdzmZyvvEU/jUVWNaMXSyx4E/25Vve6Fpq
+65s/Q3kcXTUx/bD4zfjyqzr02uNny4Op4kUAaRxdAoGBAK0t4FVXTOWIf6xxgGMZ
+BBkNu6N1n/Io3pliCpYpZrBE/6NEOaEeniyaM0BbAatR+SemaYICnEj31ueonlIs
+PLUi9H177Cnu2DZwBMaWx78r260RIxF+bcdTBjQ51fXt7/44okFjHmCcRVA1q/Rc
+WdI83+niMywS/XVfLAMhnFWR
+-----END PRIVATE KEY-----

__tests__/fixtures/normalize-manifest/throw name non-string/expected.json

@@ -1,3 +1,3 @@
 {
-  "_error": "Fatal problem with name in manifest: must be a string"
+  "_error": "\"name\" is not a string"
 }

__tests__/util/request-manager.js

@@ -0,0 +1,36 @@
+/* @flow */
+/* eslint max-len: 0 */
+
+import {NoopReporter} from '../../src/reporters/index.js';
+import Config from '../../src/config.js';
+import type {ConfigOptions} from '../../src/config.js';
+import * as fs from '../../src/util/fs.js';
+
+jasmine.DEFAULT_TIMEOUT_INTERVAL = 60000;
+
+const https = require('https');
+const path = require('path');
+
+async function createConfig(opts: ConfigOptions = {}): Promise<Config> {
+  const config = new Config(new NoopReporter());
+  await config.init(opts);
+  return config;
+}
+
+test('RequestManager.request with cafile', async () => {
+  let body;
+  const options = {
+    key: await fs.readFile(path.join(__dirname, '..', 'fixtures', 'certificates', 'server-key.pem')),
+    cert: await fs.readFile(path.join(__dirname, '..', 'fixtures', 'certificates', 'server-cert.pem')),
+  };
+  const server = https.createServer(options, (req, res) => { res.end('ok'); });
+  try {
+    server.listen(0);
+    const config = await createConfig({'cafile': path.join(__dirname, '..', 'fixtures', 'certificates', 'cacerts.pem')});
+    const port = server.address().port;
+    body = await config.requestManager.request({url: `https://localhost:${port}/?nocache`, headers: {Connection: 'close'}});
+  } finally {
+    server.close();
+  }
+  expect(body).toBe('ok');
+});

package.json

@@ -13,6 +13,7 @@
     "death": "^1.0.0",
     "debug": "^2.2.0",
     "defaults": "^1.0.3",
+    "detect-indent": "^4.0.0",
     "diff": "^2.2.1",
     "eslint-plugin-react": "5.2.2",
     "github": "2.5.1",

src/cli/commands/add.js

@@ -92,7 +92,7 @@ export class Add extends Install {
     const {dev, exact, tilde, optional, peer} = this.flags;
 
     // get all the different registry manifests in this folder
-    const jsons = await this.getRootManifests();
+    const manifests = await this.config.getRootManifests();
 
     // add new patterns to their appropriate registry manifest
     for (const pattern of this.resolver.dedupePatterns(this.args)) {
@@ -134,7 +134,7 @@ export class Add extends Install {
       }
 
       // add it to manifest
-      const object = jsons[ref.registry].object;
+      const object = manifests[ref.registry].object;
       for (const key of targetKeys) {
         const target = object[key] = object[key] || {};
         target[pkg.name] = version;
@@ -149,7 +149,7 @@ export class Add extends Install {
       this.resolver.removePattern(pattern);
     }
 
-    await this.saveRootManifests(jsons);
+    await this.config.saveRootManifests(manifests);
   }
 }
 

src/cli/commands/init.js

@@ -4,8 +4,6 @@ import type {Reporter} from '../../reporters/index.js';
 import type Config from '../../config.js';
 import {stringifyPerson} from '../../util/normalize-manifest/util.js';
 import {registryNames} from '../../registries/index.js';
-import Lockfile from '../../lockfile/wrapper.js';
-import {Install} from './install.js';
 import * as child from '../../util/child.js';
 import * as fs from '../../util/fs.js';
 
@@ -22,9 +20,7 @@ export async function run(
   flags: Object,
   args: Array<string>,
 ): Promise<void> {
-  const lockfile = new Lockfile();
-  const install = new Install(flags, config, reporter, lockfile);
-  const manifests = await install.getRootManifests();
+  const manifests = await config.getRootManifests();
 
   let gitUrl;
   const author = {
@@ -137,5 +133,5 @@ export async function run(
     reporter.success(`Saved ${path.basename(targetManifest.loc)}`);
   }
 
-  await install.saveRootManifests(manifests);
+  await config.saveRootManifests(manifests);
 }

src/cli/commands/install.js

@@ -6,7 +6,6 @@ import type {Manifest, DependencyRequestPatterns} from '../../types.js';
 import type Config from '../../config.js';
 import type {RegistryNames} from '../../registries/index.js';
 import normalizeManifest from '../../util/normalize-manifest/index.js';
-import {stringify} from '../../util/misc.js';
 import {registryNames} from '../../registries/index.js';
 import {MessageError} from '../../errors.js';
 import Lockfile from '../../lockfile/wrapper.js';
@@ -42,14 +41,6 @@ export type InstallCwdRequest = [
   Object
 ];
 
-type RootManifests = {
-  [registryName: RegistryNames]: {
-    loc: string,
-    object: Object,
-    exists: boolean,
-  }
-};
-
 type IntegrityMatch = {
   actual: string,
   expected: string,
@@ -129,14 +120,6 @@ function normalizeFlags(config: Config, rawFlags: Object): Flags {
   return flags;
 }
 
-const sortObject = (object) => {
-  const sortedObject = {};
-  Object.keys(object).sort().forEach((item) => {
-    sortedObject[item] = object[item];
-  });
-  return sortedObject;
-};
-
 export class Install {
   constructor(
     flags: Object,
@@ -445,7 +428,7 @@ export class Install {
 
     // save resolutions to their appropriate root manifest
     if (Object.keys(this.resolutions).length) {
-      const jsons = await this.getRootManifests();
+      const manifests = await this.config.getRootManifests();
 
       for (const name in this.resolutions) {
         const version = this.resolutions[name];
@@ -467,59 +450,17 @@ export class Install {
         const ref = manifest._reference;
         invariant(ref, 'expected reference');
 
-        const object = jsons[ref.registry].object;
+        const object = manifests[ref.registry].object;
         object.resolutions = object.resolutions || {};
         object.resolutions[name] = version;
       }
 
-      await this.saveRootManifests(jsons);
+      await this.config.saveRootManifests(manifests);
     }
 
     return flattenedPatterns;
   }
 
-  /**
-   * Get root manifests.
-   */
-
-  async getRootManifests(): Promise<RootManifests> {
-    const manifests: RootManifests = {};
-    for (const registryName of registryNames) {
-      const registry = registries[registryName];
-      const jsonLoc = path.join(this.config.cwd, registry.filename);
-
-      let object = {};
-      let exists = false;
-      if (await fs.exists(jsonLoc)) {
-        exists = true;
-        object = await fs.readJson(jsonLoc);
-      }
-      manifests[registryName] = {loc: jsonLoc, object, exists};
-    }
-    return manifests;
-  }
-
-  /**
-   * Save root manifests.
-   */
-
-  async saveRootManifests(manifests: RootManifests): Promise<void> {
-    for (const registryName of registryNames) {
-      const {loc, object, exists} = manifests[registryName];
-      if (!exists && !Object.keys(object).length) {
-        continue;
-      }
-
-      for (const field of constants.DEPENDENCY_TYPES) {
-        if (object[field]) {
-          object[field] = sortObject(object[field]);
-        }
-      }
-
-      await fs.writeFile(loc, stringify(object) + '\n');
-    }
-  }
-
   /**
    * Save updated integrity and lockfiles.
    */

src/cli/commands/login.js

@@ -33,12 +33,21 @@ async function getCredentials(config: Config, reporter: Reporter): Promise<?{
   return {username, email};
 }
 
-export async function getToken(config: Config, reporter: Reporter): Promise<
+export async function getToken(config: Config, reporter: Reporter, name: string = ''): Promise<
   () => Promise<void>
 > {
+  const auth = config.registries.npm.getAuth(name);
+  if (auth) {
+    config.registries.npm.setToken(auth);
+    return function revoke(): Promise<void> {
+      reporter.info(reporter.lang('notRevokingConfigToken'));
+      return Promise.resolve();
+    };
+  }
+
   const env = process.env.YARN_AUTH_TOKEN || process.env.KPM_AUTH_TOKEN || process.env.NPM_AUTH_TOKEN;
   if (env) {
-    config.registries.npm.setToken(env);
+    config.registries.npm.setToken(`Bearer ${env}`);
     return function revoke(): Promise<void> {
       reporter.info(reporter.lang('notRevokingEnvToken'));
       return Promise.resolve();
@@ -80,7 +89,7 @@ export async function getToken(config: Config, reporter: Reporter): Promise<
     reporter.success(reporter.lang('loggedIn'));
 
     const token = res.token;
-    config.registries.npm.setToken(token);
+    config.registries.npm.setToken(`Bearer ${token}`);
 
     return async function revoke(): Promise<void> {
       reporter.success(reporter.lang('revokedToken'));

src/cli/commands/owner.js

@@ -34,7 +34,7 @@ export async function mutate(
 
   const msgs = buildMessages(username, name);
   reporter.step(1, 3, reporter.lang('loggingIn'));
-  const revoke = await getToken(config, reporter);
+  const revoke = await getToken(config, reporter, name);
 
   reporter.step(2, 3, msgs.info);
   const user = await config.registries.npm.request(`-/user/org.couchdb.user:${username}`);
@@ -160,7 +160,7 @@ export const {run, setFlags} = buildSubCommands('owner', {
     const name = await getName(args, config);
 
     reporter.step(1, 3, reporter.lang('loggingIn'));
-    const revoke = await getToken(config, reporter);
+    const revoke = await getToken(config, reporter, name);
 
     reporter.step(2, 3, reporter.lang('ownerGetting', name));
     const pkg = await config.registries.npm.request(name);

src/cli/commands/pack.js

@@ -32,6 +32,7 @@ const DEFAULT_IGNORE = ignoreLinesToRegex([
   '*.swp',
   '._*',
   'npm-debug.log',
+  'yarn-error.log',
   '.npmrc',
   '.yarnrc',
   '.npmignore',