duosecurity / duo_log_sync

MIT License
37 stars 28 forks source link

Graceful shutdown #6

Open elyscape opened 4 years ago

elyscape commented 4 years ago

Currently, there doesn't appear to be a way to gracefully shutdown the app. Ideally, it would trap SIGINT and SIGTERM with a handler that finishes sending any already-retrieved log entries to the transport, updates the checkpoints files, and then terminates.

elyscape commented 4 years ago

SIGINT appears to be handled in the D194 branch in 2ef41b88c9f71bfdad825587ff0422bc6e53b03f. Ideally this would also handle SIGTERM so that this can be run as a service.

elyscape commented 3 years ago

Any chance of this getting a review? Pinging @rohanbendre @duo-kfleischman.