Skip to content
Snippets Groups Projects
Commit 63713305 authored by Sigmund, Dominik's avatar Sigmund, Dominik
Browse files

Bug fix default metrics

parent c33df380
Branches
No related tags found
1 merge request!1Master to main
This commit is part of merge request !1. Comments created here will be created in the context of that merge request.
...@@ -52,7 +52,7 @@ export class Metrics { ...@@ -52,7 +52,7 @@ export class Metrics {
this._disableDefaultMetrics = false this._disableDefaultMetrics = false
} }
if (!this._disableDefaultMetrics) { if (!this._disableDefaultMetrics) {
this._client._collectDefaultMetrics = this._client.collectDefaultMetrics this._client.collectDefaultMetrics()
} }
if (!this._disableErrorCounter) { if (!this._disableErrorCounter) {
this._numOfErrors = new this._client.Counter({ this._numOfErrors = new this._client.Counter({
...@@ -102,9 +102,6 @@ export class Metrics { ...@@ -102,9 +102,6 @@ export class Metrics {
} }
public endpoint = (req: express.Request, res: express.Response): void => { public endpoint = (req: express.Request, res: express.Response): void => {
if (!this._disableDefaultMetrics) {
this._client._collectDefaultMetrics()
}
res.set('Content-Type', this._client.register.contentType) res.set('Content-Type', this._client.register.contentType)
res.status(200) res.status(200)
res.end(this._client.register.metrics()) res.end(this._client.register.metrics())
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment