Skip to content

Improved context management#17

Open
dstarke wants to merge 1 commit intopersvr:masterfrom
Quantivo:master
Open

Improved context management#17
dstarke wants to merge 1 commit intopersvr:masterfrom
Quantivo:master

Conversation

@dstarke
Copy link

@dstarke dstarke commented Jun 3, 2011

This change fixes two issues:

  • Contexts could be leaked (through the vars variable) between requests. The vars variable is now copied instead of referenced to prevent this.
  • Also, a context that reaches the finally block and is suspendable should be suspended before the current promise context is set to null.

Ultimately, these changes provide part of the foundation for some of the fixes I will be contributing to perstore/transaction.

I and my employer Quantivo have a CLA on file.

…he current context at the conclusion of a request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant