Support for multiple accounts #2

Closed
felixge opened this Issue Oct 23, 2010 · 2 comments

Comments

Projects
None yet
2 participants
@felixge

felixge commented Oct 23, 2010

Just looked over the code, and it seems like setAuth effects the state of the module, making it impossible to work with multiple accounts. Is that correct, or are there no race conditions to be expected when calling setAuth?

@indexzero

This comment has been minimized.

Show comment
Hide comment
@indexzero

indexzero Oct 23, 2010

Member

No, this was a by design decision that I made. I've been rethinking it though as we've had some multi-account needs at Nodejitsu. Perhaps refactoring 'core.js' into 'client.js' to support these scenarios.

There would probably be race conditions from this line, since the X-AUTH-TOKEN is set from the cloudfiles.config object: http://github.com/nodejitsu/node-cloudfiles/blob/master/lib/cloudfiles/utils.js#L120

Member

indexzero commented Oct 23, 2010

No, this was a by design decision that I made. I've been rethinking it though as we've had some multi-account needs at Nodejitsu. Perhaps refactoring 'core.js' into 'client.js' to support these scenarios.

There would probably be race conditions from this line, since the X-AUTH-TOKEN is set from the cloudfiles.config object: http://github.com/nodejitsu/node-cloudfiles/blob/master/lib/cloudfiles/utils.js#L120

@indexzero

This comment has been minimized.

Show comment
Hide comment
@indexzero

indexzero Dec 25, 2010

Member

Fixed in 0.2.0

Member

indexzero commented Dec 25, 2010

Fixed in 0.2.0

This issue was closed.

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