Skip to content
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

Call interceptors don't always initialize the call context #9

Closed
yallie opened this issue Feb 17, 2017 · 1 comment
Closed

Call interceptors don't always initialize the call context #9

yallie opened this issue Feb 17, 2017 · 1 comment
Assignees
@yallie
Milestone
2.9

Comments

@yallie
Copy link
Member

yallie commented Feb 17, 2017

Reported by Max Sobolev, see commit 76c5dd8.
@yallie yallie changed the title Call interceptors don't initialize the call context sometimes Call interceptors don't always initialize the call context Feb 17, 2017
yallie added a commit that referenced this issue Feb 19, 2017
@yallie
Added a failing unit test for the call interceptor, #9.
c606f1f
yallie added a commit that referenced this issue Feb 19, 2017
@yallie
Fixed the call context initialization bug in call interceptors, #9.
4799f9a
yallie added a commit that referenced this issue Feb 19, 2017
@yallie
Cleaned up lots of extra methods in the unit test class, #9.
f490842
yallie added a commit that referenced this issue Feb 19, 2017
@yallie
Fixed messed up namespaces, #9.
98ef576
yallie added a commit that referenced this issue Feb 19, 2017
@yallie
Converted the bug demo application to the regression test and removed…
c901125 
… it as it's not really a usage example, #9.
yallie added a commit that referenced this issue Feb 20, 2017
@yallie
Simplified the unit test for the call interception bug, #9.
56ce60d
@yallie yallie self-assigned this Feb 20, 2017
@yallie yallie added this to the 2.9 milestone Feb 20, 2017
@yallie
Copy link
Member Author

yallie commented Feb 20, 2017

That's amazing how this bug hasn't been discovered until now.

@yallie yallie closed this as completed Feb 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@yallie yallie

Labels
None yet
Projects
None yet
Milestone
2.9
Development

No branches or pull requests
1 participant
@yallie