Marc Pichler
Marc Pichler
@cpatti97100 any updates on this? :slightly_smiling_face:
@cpatti97100 closing this issue, if you still run into trouble with the new unidici instrumentation, please open a new issue over at https://github.com/open-telemetry/opentelemetry-js-contrib
@biswajit-nanda we have not gotten around to work on this one yet. I'm marking it `up-for-grabs` in case someone has some cycles to work on it.
This took me quite a while to figure out today: It looks like this is already set correctly as long as the response handler subscribes to the `data` event on...
Very minimal reproduction: ```js // Will NOT add span status const req = http.get({ hostname: 'httpstat.us', port: 80, path: '/410', method: 'GET', }, (response) => { }); req.end(); // Will...
This is still happening (see reproducer). I'm unassigning myself as I won't have time to work on this for now.
@ianhomer thanks for working on this. We do realize that this is an important feature to have, however, we're currently in the process of re-writing the exporters for them to...
> I'm happy to hear that. So... What should I do next? Is it OK to merge the upstream master into this PR, then notify you for ready? Yes, merging...
@sugi any update? :slightly_smiling_face: Let me know if there's anything I can do to help this PR along :slightly_smiling_face:
> Ouch... I'm sorry. I'll update and merge upstream again this weekend. Please give time for me. No worries, this is great work - looking forward to merging this soon....