Page MenuHomePhabricator

Timeout `interpreter {{{ ... }}}` blocks after 15 seconds
ClosedPublic

Authored by epriestley on Nov 25 2013, 9:07 PM.
Tags
None
Referenced Files
F15412829: D7651.id17275.diff
Wed, Mar 19, 3:09 PM
F15397455: D7651.id17279.diff
Sun, Mar 16, 8:20 PM
F15396473: D7651.id17275.diff
Sun, Mar 16, 2:35 PM
F15395375: D7651.id.diff
Sun, Mar 16, 6:21 AM
F15393599: D7651.diff
Sat, Mar 15, 10:38 PM
F15381979: D7651.id.diff
Fri, Mar 14, 10:12 AM
F15337257: D7651.id17275.diff
Sun, Mar 9, 4:59 AM
F15308314: D7651.id17279.diff
Thu, Mar 6, 6:08 AM
Subscribers

Details

Summary

Fixes T4149. This could be a little cleaner (configurable time limits, explicit timeout errors) but stop the major case of looping/infinite commands.

Test Plan

Added sleep 5 && and set timeout to 1, saw an error + kill.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped