New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improved context management #17

Open

dstarke wants to merge 1 commit into persvr:master from Quantivo:master

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.


          fixed leaking of contexts between requests and correctly suspending t…

2fe3ae8

…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