[TIMOB-12771] CLI: Logging out after already being logged out throws error

GitHub Issue	n/a
Type	Bug
Priority	High
Status	Closed
Resolution	Fixed
Resolution Date	2013-03-04T23:25:30.000+0000
Affected Version/s	Release 3.0.0
Fix Version/s	Release 3.1.0, 2013 Sprint 05 JS, 2013 Sprint 05
Components	CLI
Labels	qe-port, qe-testadded
Reporter	Chris Barber
Assignee	Chris Barber
Created	2013-02-16T05:26:39.000+0000
Updated	2013-09-30T11:31:51.000+0000

Description

If you are already logged out, subsequent logout attempts throw the following error:

/Users/chris/appc/titanium/node_modules/node-appc/lib/analytics.js:181
			var	sessionCookie = status.cookie.match(/(PHPSESSID=[A-Za-z0-9]+)/),
			   	                              ^
TypeError: Cannot call method 'match' of undefined
    at /Users/chris/appc/titanium/node_modules/node-appc/lib/analytics.js:181:38
    at async.parallel (/Users/chris/appc/titanium/node_modules/async/lib/async.js:476:17)
    at async.forEach (/Users/chris/appc/titanium/node_modules/async/lib/async.js:94:25)
    at async.parallel (/Users/chris/appc/titanium/node_modules/async/lib/async.js:473:21)
    at process.on.async.parallel.osinfo (/Users/chris/appc/titanium/node_modules/node-appc/lib/analytics.js:56:5)
    at module.exports.getOSInfo (/Users/chris/appc/titanium/node_modules/node-appc/lib/environ.js:215:4)
    at ChildProcess.exithandler (child_process.js:538:7)
    at ChildProcess.EventEmitter.emit (events.js:99:17)
    at maybeClose (child_process.js:638:16)
    at Socket.ChildProcess.spawn.stdin (child_process.js:815:11)

This error was introduced when fixing the bug with apps not showing up in my.appcelerator.com [TIMOB-12653].

Comments

Chris Barber 2013-02-16

Master pull request: https://github.com/appcelerator/node-appc/pull/34
Shyam Bhadauria 2013-03-05

Its not giving error with the below version:- CLI - 3.0.24-cr

JSON Source