Skip to content
This repository has been archived by the owner on Mar 15, 2022. It is now read-only.

Handler now accepts customizable routes for liveness and readiness #20

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

Conversation

winhung
Copy link

@winhung winhung commented Apr 24, 2018

No description provided.

@winhung
Copy link
Author

winhung commented Apr 25, 2018

@mattmoyer any chance of reviewing this ?

@qneyrat
Copy link

qneyrat commented Oct 19, 2018

👍 for this feature ! But maybe you add new method to create handler with custom to avoid breaking change on api.
@mattmoyer: dev is freeze ?

@winhung
Copy link
Author

winhung commented Oct 19, 2018

👍 for this feature ! But maybe you add new method to create handler with custom to avoid breaking change on api.
@mattmoyer: dev is freeze ?

Ah yes....that is a better solution. I will do that.

@qneyrat
Copy link

qneyrat commented Oct 20, 2018

@mattmoyer you can review? :)

@rmasclef
Copy link

rmasclef commented Nov 9, 2018

@mattmoyer any updates on this PR ? This feature will be very much appreciated 👍

@geoah
Copy link

geoah commented Mar 19, 2019

Could we pretty please get this merged? :D

@nabadger
Copy link

Any other updates? I really want a /healthz url ;)

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.

5 participants