Skip to content

feat: add bearerToken builder to pushgateway exporter#968

Merged
dhoard merged 3 commits intoprometheus:mainfrom
FUSAKLA:fus-pushgateway-bearer-token
Jul 18, 2024
Merged

feat: add bearerToken builder to pushgateway exporter#968
dhoard merged 3 commits intoprometheus:mainfrom
FUSAKLA:fus-pushgateway-bearer-token

Conversation

@FUSAKLA
Copy link
Contributor

@FUSAKLA FUSAKLA commented Jun 18, 2024

Add possibility to set bearer token when pushingmetrics to pushgateway

The PGW itself does not support the bearer token auth but it is needed if behind some auth proxy for example

FUSAKLA added 3 commits June 18, 2024 15:10
Signed-off-by: Martin Chodur <m.chodur@seznam.cz>
Signed-off-by: Martin Chodur <m.chodur@seznam.cz>
Signed-off-by: Martin Chodur <m.chodur@seznam.cz>
@FUSAKLA FUSAKLA marked this pull request as ready for review June 18, 2024 13:43
@FUSAKLA
Copy link
Contributor Author

FUSAKLA commented Jun 18, 2024

@fstab would you PTAL? 🙏

I initially added also integration tests, but PGW itself does not support the bearer token auth yet so I dropped them since it was testing nothing.

@FUSAKLA
Copy link
Contributor Author

FUSAKLA commented Jul 18, 2024

Or maybe @dhoard ?

@dhoard dhoard merged commit 594a9de into prometheus:main Jul 18, 2024
@FUSAKLA
Copy link
Contributor Author

FUSAKLA commented Jul 18, 2024

Thanks! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants