Jitsu snapshots activate #426

Open
Swaagie opened this Issue Mar 27, 2013 · 2 comments

Comments

Projects
None yet
2 participants
Owner

Swaagie commented Mar 27, 2013

Running snapshots active against the current directory for which I was not authorized, will not show a 401. It will prompt for the snapshot name without displaying a list. Actually entering a snapshot name will cause jitsu to crash because snapshots = undefined

info:    Welcome to Nodejitsu swaagie
info:    jitsu v0.12.8, node v0.8.22
info:    It worked if it ends with Nodejitsu ok
info:    Executing command snapshots activate
info:    Attempting to load snapshots for 
info:    Listing snapshots for handbook
prompt: Snapshot Name:  4.3.5

/var/www/nodejitsu/jitsu/lib/jitsu/commands/snapshots.js:217
      var snapshot = snapshots.filter(function (snap) {
                               ^
TypeError: Object true has no method 'filter'
    at snapshots.activate (/var/www/nodejitsu/jitsu/lib/jitsu/commands/snapshots.js:217:32)
    at prompt.get (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/lib/prompt.js:317:32)
    at async.forEachSeries.iterate (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/node_modules/utile/node_modules/async/lib/async.js:116:25)
    at assembler (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/lib/prompt.js:314:9)
    at prompt.get (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/lib/prompt.js:323:32)
    at prompt.getInput (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/lib/prompt.js:598:5)
    at Interface.onLine (/var/www/nodejitsu/jitsu/node_modules/flatiron/node_modules/prompt/node_modules/read/lib/read.js:110:5)
    at Interface.EventEmitter.emit (events.js:96:17)
    at Interface._onLine (readline.js:200:10)
    at Interface._line (readline.js:518:8)
Contributor

blakmatrix commented Mar 27, 2013

When I attempted to reproduce this on Mac OSX I got:

$ jitsu snapshots activate                                                               [12:40:52]

node.js:537
    var cwd = process.cwd();
                      ^
Error: EACCES, permission denied
    at Function.startup.resolveArgv0 (node.js:537:23)
    at startup (node.js:55:13)
    at node.js:628:3

I arrived at this by making a directory set its permissions to 600 and then changed the owner to root.
@Swaagie How do you reproduce your example?

Owner

Swaagie commented Mar 27, 2013

@blakmatrix I (accidently) ran the command while my pwd was handbook, e.g. I was trying to list the snapshots of a nodejitsu app without being actually authenticated to do so. I'm normally logged under my own account and use -j to switch to nodejitsus'

Normal users with multiple accounts might think something is wrong with jitsu while in reality they are just not authenticated for their current app.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment