search

cmanzana / node-publish

MIT License
13 stars 7 forks source link

Broken in Travis #7

Open Cu3PO42 opened 9 years ago

Cu3PO42 commented 9 years ago

If running in Travis, a runtime error occurs, specifically:

assert.js:93
  throw new assert.AssertionError({
        ^
AssertionError: must pass params to adduser
    at CachingRegistryClient.adduser (/home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/node_modules/npm/node_modules/npm-registry-client/lib/adduser.js:8:3)
    at npmAddUser (/home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/index.js:111:18)
    at /home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/index.js:59:29
    at /home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/index.js:34:13
    at /home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/node_modules/npm/lib/view.js:144:28
    at saved (/home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/node_modules/npm/lib/cache/caching-client.js:173:7)
    at /home/travis/build/Cu3PO42/KeySAVCoreJS/KeySAVCoreJS/scripts/node_modules/publish/node_modules/npm/node_modules/graceful-fs/polyfills.js:210:7
    at Object.oncomplete (fs.js:108:15)

What appears to have happened is that the API changed here. According to https://www.npmjs.com/package/npm-registry-client you'd pass in something along the lines of {auth: npmUser}.

I would also like to request support for authentication via auth tokens.

Workaround

- echo -e "$NPM_USERNAME\n$NPM_PASSWORD\n$NPM_EMAIL" | npm login
- unset TRAVIS
- publish