sendnrw/windows_exporter
2
0
Fork 0
mirror of https://github.com/prometheus-community/windows_exporter.git synced 2026-03-02 08:36:36 +00:00
Code Issues Packages Projects Releases Wiki Activity

Explicitly use first ADFS result

Browse Source 
Perflib ADFS only returns a single data result, so looping over data is
unnecessary
This commit is contained in:
Ben Reedy
2019-09-23 19:24:48 +10:00
parent 2f0a57898f
commit 29b020999d
1 changed files with 55 additions and 57 deletions
Download Patch File Download Diff File Expand all files Collapse all files

112
collector/adfs.go
View File

@@ -119,72 +119,70 @@ func (c *adfsCollector) Collect(ctx *ScrapeContext, ch chan<- prometheus.Metric)
return err return err
} }
for _, adfs := range adfsData { ch <- prometheus.MustNewConstMetric(
ch <- prometheus.MustNewConstMetric( c.adLoginConnectionFailures,
c.adLoginConnectionFailures, prometheus.CounterValue,
prometheus.CounterValue, adfsData[0].AdLoginConnectionFailures,
adfs.AdLoginConnectionFailures, )
)
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.certificateAuthentications, c.certificateAuthentications,
prometheus.CounterValue, prometheus.CounterValue,
adfs.CertificateAuthentications, adfsData[0].CertificateAuthentications,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.deviceAuthentications, c.deviceAuthentications,
prometheus.CounterValue, prometheus.CounterValue,
adfs.DeviceAuthentications, adfsData[0].DeviceAuthentications,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.extranetAccountLockouts, c.extranetAccountLockouts,
prometheus.CounterValue, prometheus.CounterValue,
adfs.ExtranetAccountLockouts, adfsData[0].ExtranetAccountLockouts,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.federatedAuthentications, c.federatedAuthentications,
prometheus.CounterValue, prometheus.CounterValue,
adfs.FederatedAuthentications, adfsData[0].FederatedAuthentications,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.passportAuthentications, c.passportAuthentications,
prometheus.CounterValue, prometheus.CounterValue,
adfs.PassportAuthentications, adfsData[0].PassportAuthentications,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.passiveRequests, c.passiveRequests,
prometheus.CounterValue, prometheus.CounterValue,
adfs.PassiveRequests, adfsData[0].PassiveRequests,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.passwordChangeFailed, c.passwordChangeFailed,
prometheus.CounterValue, prometheus.CounterValue,
adfs.PasswordChangeFailed, adfsData[0].PasswordChangeFailed,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.passwordChangeSucceeded, c.passwordChangeSucceeded,
prometheus.CounterValue, prometheus.CounterValue,
adfs.PasswordChangeSucceeded, adfsData[0].PasswordChangeSucceeded,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.tokenRequests, c.tokenRequests,
prometheus.CounterValue, prometheus.CounterValue,
adfs.TokenRequests, adfsData[0].TokenRequests,
) )
ch <- prometheus.MustNewConstMetric( ch <- prometheus.MustNewConstMetric(
c.windowsIntegratedAuthentications, c.windowsIntegratedAuthentications,
prometheus.CounterValue, prometheus.CounterValue,
adfs.WindowsIntegratedAuthentications, adfsData[0].WindowsIntegratedAuthentications,
) )
}
return nil return nil
} }