Skip to content
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

chore: add metrics config enabled #138

Merged
merged 13 commits into from
Feb 18, 2025
Merged

Conversation

RafilxTenfen
Copy link
Contributor

@RafilxTenfen RafilxTenfen commented Feb 17, 2025

  • Add new config that sets if metrics should be enabled (disabled by default)

@RafilxTenfen RafilxTenfen linked an issue Feb 17, 2025 that may be closed by this pull request
1 task
@RafilxTenfen RafilxTenfen marked this pull request as ready for review February 17, 2025 22:55
@RafilxTenfen RafilxTenfen changed the title fix: authenticate prometheus chore: add metrics config enabled Feb 18, 2025
@RafilxTenfen
Copy link
Contributor Author

rollback refactory as the security fixes should be to minimum

@RafilxTenfen RafilxTenfen merged commit 6ba1796 into main Feb 18, 2025
15 checks passed
RafilxTenfen added a commit that referenced this pull request Feb 18, 2025
* chore: add optio to enable metrics and set disabled by default

* chore: add jwt auth to metrics endpoint

* chore: add cli to generate metrics jwt token for testing

* chore: add test for JWT auth

* fix: lint stutters

* chore: add #138 to cl

* fix: lint

* chore: removed JWT authentication in metrics

* chore: update cl

* chore: removed unused cosnt

* chore: removed JWT auth const

* go mod tidy

* chore: rollback admin refactory
RafilxTenfen added a commit that referenced this pull request Feb 18, 2025
* chore: add metrics config enabled (#138)

* chore: add optio to enable metrics and set disabled by default

* chore: add jwt auth to metrics endpoint

* chore: add cli to generate metrics jwt token for testing

* chore: add test for JWT auth

* fix: lint stutters

* chore: add #138 to cl

* fix: lint

* chore: removed JWT authentication in metrics

* chore: update cl

* chore: removed unused cosnt

* chore: removed JWT auth const

* go mod tidy

* chore: rollback admin refactory

* chore: remove #114 from cl
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.

Fix unauthenticated Prometheus server
3 participants