un-hardcode vault token monitoring in watcher
- Dominant language
- Go
- Stars
- 96
- Forks
- 13
- PR merge metrics
- No merged PRs in 30d
Description
The watcher has a hardcoded field for monitoring the vault-token as part of its process. This means for testing the vault-token you need to spin up a watcher with that config.
Seems like it would be better to integrate it without the hardcoding. In a way that would be easier to test.
I've refactored this already in consul-template and should use that as a guide to convert here after the non-template notifiers work.
Contributor guide
No contributing guide indexed for this repository
Research direction
Start with the watcher’s hardcoded vault-token monitoring and compare the existing refactor in consul-template. Check how this fits with the non-template notifiers; done means vault-token monitoring is integrated without hardcoding and can be tested without starting a watcher with that configuration.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 42/100