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

Add optional debugger logging #271

Closed
mousetraps opened this issue Jul 10, 2015 · 2 comments
Closed

Add optional debugger logging #271

mousetraps opened this issue Jul 10, 2015 · 2 comments
Assignees
@mousetraps
Labels
debugging feature fun-size in-progress
Milestone
July 2015

Comments

@mousetraps
Copy link
Contributor

This will help us track down race various debugging oddities that people are running into.
@kant2002
Copy link
Contributor

Could you give more information about areas where debugger logging could be added?

@mousetraps
Copy link
Contributor Author

Sure thing! Turns out we already log all the info we need when the debugger is attached (it all passes through AD7Engine.DebugWriteLine)
https://github.com/Microsoft/nodejstools/blob/master/Nodejs/Product/Nodejs/Debugger/DebugEngine/AD7Engine.cs#L1293

So it would be helpful to be able to optionally save this out to a log file rather than it only being available when a debugger is attached. That way when people run into an issue, we can ask them to turn on logging, and when they send it to us, we can just do a diff their machine's output with what what we might expect it to be.

@mousetraps mousetraps mentioned this issue Jul 14, 2015
Closed
@mousetraps mousetraps modified the milestone: July Jul 24, 2015
@mousetraps mousetraps self-assigned this Jul 26, 2015
mousetraps added a commit to mousetraps/nodejstools that referenced this issue Aug 3, 2015
@mousetraps
microsoft#271 Add optional debugger logging
68c9198 
- Added live diagnostics log pane
- Added new live diagnostic option
- enable live diagnostics on debugger cmds and events
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mousetraps mousetraps

Labels
debugging feature fun-size in-progress
Projects
None yet
Milestone
July 2015
Development

No branches or pull requests
2 participants
@mousetraps @kant2002