Peter McAtominey

Results 6 comments of Peter McAtominey

Was going to jump on this, just wondering if it should be a separate command or rolled into the monitor command thats in progress?

Updated to switch test from `SIGINFO` to `SIGHUP` as the former does not exist on linux/amd64.

@natefinch thanks for the review. I've updated the code to apply a timeout after a SIGINT and tests to cover this, aswell as testing that a deferred function call within...

Just wanted to check if anyone is working on this before opening a PR.

Hi @flowchartsman, My PR will allow the magefile to catch a sigint, this is covered with [this test](https://github.com/magefile/mage/pull/313/files#diff-33b3620f23e56234e6d150c8f95063354ad0cc3b7d48ffc4aaedfa750180e995R23), sigkill is issued after a second sigint is received.

Traced the issue to #14269, the `result` is never updated [here](https://github.com/hashicorp/vault/pull/14269/files#diff-aa9bfd1a638fbb706f8e8920297902937011160319d9679add5dca56e5ab8277R1351) with the response of the redirected request.