-
Notifications
You must be signed in to change notification settings - Fork 380
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
Pushgateway call registry async metrics methods #390
Comments
Yes, I think you're right. I missed two high-level functions evidently :(. PR welcome, otherwise I'll fix it in the next few days. Thanks for finding that. |
Hey guys @zD98 @zbjornson. I have just met the same bug. Do you have any plans to merge PR and release a new version? |
This issue still occurs at version 13.Please publish a new sub-version to resolve it, thanks. |
This was fixed in #419 (thanks @adorin). Until the new version is published, you can install from github using |
v13.1.0 is now published with this fix! |
prom-client/lib/pushgateway.js
Line 78 in 25f9c40
metrics
is an async method.So should calling
metrics
withasync/await
?The text was updated successfully, but these errors were encountered: