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.
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
[1.4.2] While debugging server, node-inspector debug console is unusable #7991
Comments
I am also getting this exact same error after the 1.4.2 update. |
I am experiencing the same issue. EnvironmentWindows 7 |
What "debug console" are you talking about? Do you mean |
@benjamn I believe he is refering to using That is what happens to me. |
Sorry. I am using node-inspector which pops up a chrome window. This error shows up in the console window of Node-inspector:
And the console is unusable. If I type in a variable to inspect it simply does not work. This is impacting debugging fairly significantly. |
@benjaminn: Does |
@brucejo75 it gets written just above that list ( |
For what it's worth, we are using the latest version of https://www.npmjs.com/package/node-inspector (0.12.8). |
Yeah, I upgraded everything. But the console is still dead. Any idea why Node-inspector is looking for the map of |
I just installed 1.4.2.1 and verified the issue is still there in that version. Repro
Result
|
OK, I found this: node-inspector/node-inspector#903 Seems that node-inspector is having a sync problem with Chrome and this bug cropped up with new versions of Chrome. Not related to Meteor... Closing, sorry for the distraction. |
All good! Obviously |
Note that this issue describes three phenomena:
Point 3 is indeed a generic issue between Point 1 and 2 are connected and actually do only occur after Meteor 1.4.2. I've tested Meteor 1.4.1.3 and there the source tree loads fine and there is no error message about |
@AronVanAmmers First, please try resetting your dev environment with |
@abernix that's encouraging, thanks 😄. Will clean up and report back. |
BTW: I do not think node-inspector will ever get fixed anymore. As of Node v6.3.0+, node applications can be debugged with the chrome debug tools directly: See "Debugging Node.js with Chrome DevTools" YAY! Looking forward to the next version of Meteor with support for Node v6. I think it is coming soon. |
@brucejo75 thanks, that's very helpful to know! Looking forward to v6. As it turns out, I was wrong on point 1 above. Indeed loading the source tree works fine in 1.4.2.3, as does the rest of the debugging functionality. The error message about |
Here's a one-liner to apply the workaround from node-inspector/node-inspector#951 and fix the behaviour of the enter key in Chrome >= 53. This adapts the file of
Note that there appears to be an issue with the cursor keys as well, and node-inspector has more issues. But with this fix basic debugging is possible. UPDATE: the original version of this message was missing some characters at the end, sorry about that! |
@AronVanAmmers - tried running from terminal and it acted like I was missing a quote or something, so i modified the file directly:
|
Thanks for the pointer, but that fix does not work for me. I did these steps:
Went to a console line I still have the enter problem? |
@brucejo75 Hm..by rebuild do you mean letting the project auto-rebuild after a change or completely stopping the app and running Also double check that your project is running on the same meteor release as the subdirectory of the file you changed. I have two meteor version directories for different projects - 1.3.3 & 1.4.2.1 - and had to make the change in both. |
@mikepaszkiewicz, thanks for the reply. I am seeing a problem, maybe with my build?
And here is the code in the file I changed:
While running node-inspector, I hit Also, how do you fire up a debug session? Just navigate to http://127.0.0.1:8080/?port=5858? |
@brucejo75 @mikepaszkiewicz there was a typo in my initial message about the one-liner. Corrected, sorry about that. I can confirm that breakpoints work for me. Indeed start by navigating to http://localhost:8080/?port=5858. |
@AronVanAmmers : I made the changes as suggested by @brucejo75 on Dec 13, 2016 and I am still getting the same issue. Any ideas on what I might be doing wrong? |
@ajitgoel I figured out my issue. I had installed node-inspector globally, not in my meteor project. So I made the requisite changes in the global repository for node-inspector. In my case that is on windows and my installation is here: C:\Users\Bruce\AppData\Roaming\npm\node_modules\node-inspector. Maybe others got tripped up by this too? |
Hi @brucejo75, |
What happens when you type |
I get the following when I run the where node-debug and where node-inspector commands respectively. C:\Users\u6041471\AppData\Roaming\npm\node-inspector When I run the npm install -g node-inspector command, I see the following error: 23885 verbose stack Error: v8-debug@0.7.7 install: |
OK, good information. Couple of things:
After all that...Now you can make the patch to Hope this helps. |
I'm glad this thread is here but this is nuts that it has been an issue almost for 5 months. I guess either no one is debugging their apps or maybe many people aren't using Meteor? Is there an alternative debugging method that everyone is using other than |
@codehatcher This issue is closed. Meteor apps can be debugged by running Edit: You really shouldn't need to install |
@abernix what was the solution that closes this issue that I also see. The highly reacted ones are the "hacks" vs official fix |
@codehatcher I guess the resolution was that there is a bug in The reproduction above still appears a bit incorrect as it doesn't show any attaching of Meteor to the debug interface – or more correctly, using But again, if you're encountering this problem, can you please open a new issue with a clear reproduction? 😄 |
repro using Regardless there is no bug in Meteor. Node-inspector had a bug and a published workaround here.. After closing the bug, this thread published instructions on how to patch your version of meteor's node-inspector. Those instructions did not work for me. Then I realized that my debug environment uses my installed version of node-inspector and I do not use the version that comes with Meteor so I published instructions on how to find the file to patch for that circumstance. @abernix is right, I am confusing things by not using hmm. thinking about it... would it be reasonable for Meteor to apply the patch as part of building the dev bundle? |
I'll open up a new issue, I am having the same issue as the person that posted in November but it seems that's buried in the comments |
The bug reported in bullet-point 3 in December is actually the same as the original issue and the subject of this issue. This is absolutely a bug with |
EDIT by @abernix: This is a
node-inspector
bug see this workaround provided by @AronVanAmmers below!.This is new with 1.4.2: I am seeing this error when loading a server debug session
I do not know if it is related, but any commands in the console get lost in space (I assume the debugger may be parsing the entire symbol tree looking for things?). The console is unusable.
This may be related to #7977
Environment
Windows 10: 1511 (OS Build 10586.633)
Meteor: 1.4.2
Chrome: 54.0.2840.71
The text was updated successfully, but these errors were encountered: