Skip to content
This repository has been archived by the owner on Jun 2, 2024. It is now read-only.

sync by install no need session #274

Closed
dead-horse opened this issue Mar 15, 2014 · 2 comments
Closed

sync by install no need session #274

dead-horse opened this issue Mar 15, 2014 · 2 comments
Assignees
Labels
Milestone

Comments

@dead-horse
Copy link
Member

in the latest npm client, every update request will has a query write=true. so it is easy to separate unpublish and install.

only need to make sure all the unpublish with the latest npm client.

@dead-horse dead-horse self-assigned this Mar 15, 2014
fengmk2 added a commit that referenced this issue Mar 16, 2014
remove session access in /name and /name/version, fixed #274
@fengmk2 fengmk2 added this to the v0.3 koa milestone Mar 16, 2014
@fengmk2
Copy link
Member

fengmk2 commented Mar 16, 2014

[email protected]+ will contains ?write=true, which include in [email protected]+ and [email protected]+

@dead-horse
Copy link
Member Author

$ npm unpublish --verbose connect -f
npm info it worked if it ends with ok
npm verb cli [ 'node',
npm verb cli   '/Users/deadhorse/node_modules/.bin/npm',
npm verb cli   'unpublish',
npm verb cli   '--verbose',
npm verb cli   'connect',
npm verb cli   '-f' ]
npm info using [email protected]
npm info using [email protected]
npm WARN using --force I sure hope you know what you are doing.
npm verb url raw connect?write=true
npm verb url resolving [ 'https://registry.npmjs.org/', './connect?write=true' ]
npm verb url resolved https://registry.npmjs.org/connect?write=true
npm info trying registry request attempt 1 at 13:16:44
npm http GET https://registry.npmjs.org/connect?write=true

we only need to make all admin users use [email protected]+, so all the update request will have ?write=true

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

No branches or pull requests

2 participants