Skip to content
This repository has been archived by the owner on Aug 2, 2024. It is now read-only.

Add ctx to CurrentHealth() monitor method #3

Open
erodrigufer opened this issue Oct 20, 2022 · 0 comments
Open

Add ctx to CurrentHealth() monitor method #3

erodrigufer opened this issue Oct 20, 2022 · 0 comments
Labels
enhancement New feature or request low priority Issue is not pressing

Comments

@erodrigufer
Copy link
Owner

https://github.com/erodrigufer/CTForchestrator_priv/blob/a71fcf4e91e693dd98aa60fea1bf25c94eef09ad/cmd/ctfsmd/monitor.go#L36

  • Adding a context will ensure a more proper shutdown of the monitor method.
@erodrigufer erodrigufer added the enhancement New feature or request label Oct 20, 2022
@erodrigufer erodrigufer added the low priority Issue is not pressing label Nov 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request low priority Issue is not pressing
Projects
None yet
Development

No branches or pull requests

1 participant