-
Notifications
You must be signed in to change notification settings - Fork 9
UnexpectedContextValue Errors when using "redis" #108
Comments
Thanks @BlueHatbRit. Let me know if you get a repro case. |
Hey @BlueHatbRit , wondering if you can try something out for me. We have a new build on our "alpha feed". It's early & hasn't been fully tested, but it has a change that may solve your errors. To try it out, you'll need to do the following:
After the above, start your app with the following comand-line params:
or
Let me know if you're able to try above out on your app, and if have a better experience here. |
Hi @mike-kaufman, sorry I've not been able to whip you up a repository which displays the issue. I'm in some final deadlines at university and I'm totally slammed. I'll give this a go for you now and let you know how it goes! |
@mike-kaufman I followed through the steps and managed to install 0.21.1 just fine, when starting up with: node index.js --glimpse.contextmanager zones I got the following stack trace when hitting my index page.
Now this seems to be something completely different coming from my express session handlers but the line causing an issue is |
Great, thanks @BlueHatbRit . Do you still see those /cc @philliphoff - FYI if this look familiar at all wrt hud/script injection? |
@mike-kaufman I can't get that far as requesting any endpoint crashes the application before glimpse reports anything. |
I'd be happy to provide you guys access to my project with a Vagrant environment if that would be useful to you. |
This would be great! You can send mail to {first}.{last}@microsoft.com. Thanks!!! |
@mike-kaufman I've added you to both repos and sent a full email which should get you going really quickly. |
@BlueHatbRit - thanks! Looking into this now. |
I am on the latest node and latest express versions and am getting this problem as well, but not only for onResponseEnd but also for onResponseSend. |
@BlueHatbRit - We think we've got your lost context issues fixed. We should have a release up on npm late next week, or you can try out the latest build on our alpha feed:
Then launch your app normally:
Let us know if you have any more lost context errors. |
@japrescott - it's quite possible that your lost context errors are a different root cause than those reported by @BlueHatbRit. I've opened #114 to track the issue you reported. What's happening here is we need to track state across async code transitions, which means we need to effectively proxy these continuation points in lots of different client libraries. We likely need to correctly proxy socket.io and/or sticky sessions. |
redis
redis
@mike-kaufman I'll give this a go now and report back. |
@mike-kaufman these issues seem to be resolved now using |
Hi @BlueHatbRit We'd love to have a chat with you about how you're using Glimpse, what's useful and what isn't. It shouldn't take more than 30 mins and there's no need to prepare in advance. If you're interested, just pick a time slot over at https://calendly.com/nikmd23/30min Thanks! |
Hi @nikmd23, I'd love to but I'm afraid I'm heading to Hong Kong for a week tomorrow and won't be around until about the Friday after I get back. Am I able to pick a slot just after then? |
Of course! Enjoy your trip and we'll chat later. |
I'm getting an error come up very frequently and I'm not sure if it's related as I'm using Bluebird as discussed above. It seems as if every HTTP request that should be logged to the console is generating one of these errors, except requests to the glimpse API.
Note: the history in the glipse UI seems to be coming through just fine.
Error
More full log sample
This is a full log sample of me loading up a few different pages around my application. As you can see, requests to glimpse seem to be logged just fine but requests to my application aren't registering.
Further details
This may be linked to the usage of Bluebird as described in #97 as it seems to be talking about the context. I'm using glimpse in a project with Bookshelf and Bluebird (npm modules) which kicked up a fuss in that last issue. However this may of course be completely unrelated.
I'll spend tomorrow morning trying to come up with a small repo to recreate the issue.
The text was updated successfully, but these errors were encountered: