Skip to content
This repository has been archived by the owner on Dec 7, 2020. It is now read-only.

[KEYCLOAK-9379] Health, metrics and profiling on separate port #464

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

fredbi
Copy link
Contributor

@fredbi fredbi commented Mar 9, 2019

Added option to serve health, metrics and profiling endpoints on separate port

  • Added support for http scheme on health & metrics endpoint
  • Added support for specific TLS config for health & metrics endpoint(including client cert for mutual TLS)
  • Updated go-chi router dependency to actual repo (not link)
  • Updated dependencies: replaced outdated repo location for go-chi/chi router

Signed-off-by: Frederic BIDON frederic@oneconcern.com

  • Removed logging from admin handler

admin_test.go Outdated Show resolved Hide resolved
admin_test.go Outdated Show resolved Hide resolved
admin_test.go Outdated Show resolved Hide resolved
@fredbi fredbi changed the title Admin listener rh [KEYCLOAK-9379] Added option to server health, metrics and profiling Mar 9, 2019
@fredbi fredbi changed the title [KEYCLOAK-9379] Added option to server health, metrics and profiling [KEYCLOAK-9379] Added option to serve health, metrics and profiling Mar 9, 2019
Added option to serve health, metrics and profiling endpoints on separate port (with https or http)

    Added support for http scheme on health & metrics endpoint
    Added support for specific TLS config for health & metrics endpoint(including client cert for mutual TLS)
    Updated go-chi router dependency to actual repo (not link)
    Updated dependencies: replaced outdated repo location for go-chi/chi router
    Removed logging from admin handler

Signed-off-by: Frederic BIDON frederic@oneconcern.com
@fredbi fredbi changed the title [KEYCLOAK-9379] Added option to serve health, metrics and profiling [KEYCLOAK-9379] Health, metrics and profiling on separate port Mar 11, 2019
@abstractj abstractj self-assigned this May 15, 2019
@abstractj abstractj removed their assignment Aug 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants