Todd Beets

@dmpriso Sorry for delay in PR response. One ask for option name change plus the test coverage ask is all that is required.

2.10.x a good target release. @derekcollison @dmpriso Are you able to make the requested updates? Once those are in we can re-base on Dev branch (2.10.x train) and merge the...

@dmpriso Review comment on `opts.go`: > Change to CertMatchSkipInvalid and cert_match_skip_invalid to better scope to Cert Store feature and matching that happens there (vs. a general feature that works with...

Thanks @dmpriso . I can see that you made the option name changes. The unit test you wrote initially for the new skip feature doesn't actually start the embedded server...

The server does process the config file at startup, but Windows store interaction happens subsequent to that in server startup flow and before listeners are enabled. I hear you on...