-
Notifications
You must be signed in to change notification settings - Fork 792
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
express plugin: multiple .end() call & .setStatus after .end() of span #908
Comments
Guess thats coming from the fact that i'm trying to guess if one layer called the next callback or not (here). I will look into it tomorrow if no one does it before |
Yes, looks like due to: opentelemetry-js/packages/opentelemetry-plugin-express/src/express.ts Lines 210 to 212 in f695e40
|
This has landed on the contrib repo, release pending open-telemetry/opentelemetry-js-contrib#6 |
Please answer these questions before submitting a bug report.
What version of OpenTelemetry are you using?
0.5.2
What did you do?
Run express example.
What did you expect to see?
Execute without
error
/warn
.What did you see instead?
The text was updated successfully, but these errors were encountered: