Skip to content
This repository has been archived by the owner on Jul 28, 2023. It is now read-only.

unable to debug worker threads #277

Closed
dnalborczyk opened this issue Jul 1, 2019 · 4 comments · Fixed by #281
Closed

unable to debug worker threads #277

dnalborczyk opened this issue Jul 1, 2019 · 4 comments · Fixed by #281
Labels
enhancement New feature or request

Comments

@dnalborczyk
Copy link

I'm unable to debug worker threads, ref: https://nodejs.org/dist/latest-v12.x/docs/api/worker_threads.html

are there any steps I'm missing to debug a worker thread?

Steps to reproduce

Tell us about your environment:

  • ndb version: v1.0.52
  • Platform / OS version: macOS Mojave 10.14.6 Beta
  • Node.js version: v12.5.0

What steps will reproduce the problem?

Please include code that reproduces the issue.

// index.js
const Worker = require('worker_threads');

const worker = new Worker('./script.js', { data: ... });
// script.js
debugger // not being hit

console.log('foo')
@alexkozy alexkozy added the enhancement New feature or request label Jul 1, 2019
@alexkozy
Copy link
Contributor

alexkozy commented Jul 1, 2019

The feature is not implemented yet but it is coming - should be available by the end of this week.

@dnalborczyk
Copy link
Author

awesome, thanks for the update @ak239 !!

alexkozy added a commit that referenced this issue Jul 3, 2019
It is full support: debugging and profiling works with Node 12.

Fixes #277
alexkozy added a commit that referenced this issue Jul 3, 2019
It is full support: debugging and profiling works with Node 12.

Fixes #277
alexkozy added a commit that referenced this issue Jul 7, 2019
It is full support: debugging and profiling works with Node 12.

Fixes #277
@alexkozy
Copy link
Contributor

alexkozy commented Jul 8, 2019

Support for workers available in ndb@1.1.0 so ndb is the only one node debugger in a wild that supports both - child processes and workers. At the same time workers are not 100% ready - they require node fix - nodejs/node#28528 - should be available soon.

@dnalborczyk
Copy link
Author

nice!!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants