search

letsencrypt / ct-woodpecker

A tool to monitor a certificate transparency log for operational problems
Mozilla Public License 2.0
182 stars 19 forks source link

Wire inclusionChecker into ct-woodpecker #42

Closed cpu closed 6 years ago

cpu commented 6 years ago

In 6313966519044023cdf4ff6ef75cebf442b06094 we added an inclusionChecker to the monitor package conditionally based on whether InclusionOpts are provided in the MonitorOptions provided to monitor.New: https://github.com/letsencrypt/ct-woodpecker/blob/6313966519044023cdf4ff6ef75cebf442b06094/monitor/monitor.go#L180-L205

We missed wiring the ability to configure InclusionOpts in the MonitorOptions that woodpecker.New calls for each of the logs in the ct-woodpecker config: https://github.com/letsencrypt/ct-woodpecker/blob/6313966519044023cdf4ff6ef75cebf442b06094/woodpecker/woodpecker.go#L244-L271

We should fix that so that the command-line tool can configure & use the inclusion checker.

cpu commented 6 years ago

Also the DBURI