The constructed mercurial url is wrong - it creates urls like
"repo/rev3456fg", which just get redirected to the default log, rather
than the commit. The url should be "repo/rev/3456fg".

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

This does not change anything for tinyurl (the default), but allows
using http://tinyarro.ws/api-create.php?utfpure=1&url=


In case invalid characters are encountered, fall back to str.

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

AI0867 reported this was causing a rare hang during disconnect.
Also, set the connection status to expired if the thread terminates
due to exception.

Boom!  Blew up, first time tested.

Alas, the fix violates RFC2812.

Usage: outcmd|irk target -
Sending "-" is still possible by echo -|irk target -

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

This fixes a corner case where if you had a non-ASCII character in a
commit and a filter command, you would end up with an unicode string and
irkerhook.py would crash.
But it would somehow work if you did not have a filter command.
For both modes cases to work, forcing unicode in do() and then using
unicode methods are required.
JSON is unicode by default, and this is why the issues happened.

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

Signed-off-by: Eric S. Raymond <esr@thyrsus.com>

It seems to collide, hard, with the (necessary) library mutex.
Also it was really complicating testing.