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

Add MySQL support to ct-woodpecker. #84

Closed jsha closed 5 years ago

jsha commented 5 years ago

Fixes #82 (we think there may be a crashing bug in the sqlite3 driver).

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 615


Changes Missing Coverage Covered Lines Changed/Added Lines %
monitor/monitor.go 21 29 72.41%
<!-- Total: 23 31 74.19% -->
Totals Coverage Status
Change from base Build 571: 0.4%
Covered Lines: 836
Relevant Lines: 1225

💛 - Coveralls
rolandshoemaker commented 5 years ago

Coveralls doesn't like reductions in coverage.

jsha commented 5 years ago

Sounds reasonable. I'll pull it out. I also remembered we should pull in the DB password from a separate file; I'll add that.