Skip to content
This repository has been archived by the owner on Jul 14, 2021. It is now read-only.

sensu-go-check: Add validate_certs, cross-link docs, and more validation #106

Merged
merged 9 commits into from
May 19, 2019

Conversation

jaredledvina
Copy link
Owner

  • Adding back validate_certs for folks using HTTPS but without a proper certificate (although, you should fix that :) )
  • Add seealso to link to the upstream check docs which cover all of the parameters here in much greater detail.
  • Add initial custom validation for the key/value pairs for annotations and labels. I still have to validate the regex for the labels keys but, this is a start.
  • Simplify the creation of the diff dictionary.

Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
@jaredledvina jaredledvina added the enhancement New feature or request label May 19, 2019
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
Signed-off-by: Jared Ledvina <jared@techsmix.net>
@jaredledvina jaredledvina changed the title Feature: sensu-go-check improvements round 4 sensu-go-check: Add validate_certs, cross-link docs, and more validation May 19, 2019
@jaredledvina jaredledvina merged commit 76a7120 into master May 19, 2019
@jaredledvina jaredledvina deleted the feature/sensu-go-check-more-bits branch May 19, 2019 15:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant