irkerd: Replace 'fallback' global with local 'nick_needs_number'

Using the new kwargs handling to pass the data through Irker() down to
Connection().

irkerd: Add kwargs handling to pass data to IRCServerConnection.connect

This makes it easy to pass data down the stack:

  Irker() (stored in Irker.kwargs)
  `-- Irker.handle() -> Dispatcher() (stored in Dispatcher.kwargs)
      `-- Dispatcher.dispatch() -> Connection() (stored in Connection.kwargs)
          `-- Connection.dequeue() -> IRCServerConnection.connect()

You can easily add data at every point in the stack (e.g. we add
'target' in Irker.handle()) and pull it back out when that's
appropriate (e.g. we tap 'target' back out in Connection()).  With
this setup we can reduce the number of global variables currently in
use, because it will be easy to pass data like passwords,
nickame-fallback-ness, etc. down to the appropriate level, without the
intermediate levels needing any changes.

irkerd: Add Target.__str__ for pretty-printing targets in log messages

Prefer the servername, falling back to the URL, falling back to
Target.__repr__().

irkerd: Convert to Python's logging module

Instead of using the local IRCClient.debug() method, use the more
flexible standard library logger.  This makes it easy to log to
syslog, rotating files, etc, using the usual logging Handlers.  The
mapping from the old implementation to the new implementation is:

  IRCClient.debug(1, message)  -> LOG.info(message)
  IRCClient.debug(2, message)  -> LOG.debug(message)
  IRCClient.debug(50, message) -> LOG.debug(message)
  Irker.logerr(errmsg)         -> LOG.error(message)

with the exception of the failed-message error, which is logged as
LOG.warning().  I didn't try and recategorize the other message log
levels, although I think a number of info-level log messages should
really be debug-level log messages.

To set the log level, the -d option now takes string arguments
(e.g. 'info', 'debug') instead of numeric arguments (e.g. '1', '2').
This breaks backward compatibility, but I think it makes the argument
more user-friendly.  If you try and set an invalid level, there's a
helpful error message to guide you in the right direction.

I also use format_exc() in Connection.dequeue (following the existing
example deeper in the Connection.dequeue nest).  The log level should
decide whether the traceback is printed or not, not whether the
exception should be raised or ignored.

irkerd: Split imported modules onto their own lines

Following PEP 8 [1]:

  Imports should usually be on separate lines, e.g.:

    Yes: import os
         import sys

    No:  import sys, os

This also makes it easier to read diffs that add and remove imports,
since you won't need a word-diff to see exactly what changed.

[1]: http://legacy.python.org/dev/peps/pep-0008/#imports

Conflicts:
irkerd

irkerd: Pull request-parsing out into Irker._parse_request

There is a lot of error checking here, which is good, but it distracts
from the core logic of Irker.handle.  By pulling the parsing out into
a private helper function, we isolate the code focused on parsing and
error checking from the code focused on dispatching and connection
management, making both easier to read.

I've also changed the Target-validation logic.  The old Target.valid
returned True if the Target URL was valid, and False otherwise.  The
new Target.validate returns None, and raises an InvalidRequest
exception with an error message describing exactly why the URL is
invalid.  We print these messages when dropping server URLs in
Irker._parse_request, while the old Irker.handle code silently dropped
invalid targets.  We also continue processing other server URLs after
an invalid Target, while the old Irker.handle code bailed out after
the first invalid Target.  Besides making the invalid URLs more
obvious in the logs and increasing resiliency to invalid URLs, these
changes allow us to pull the URL-to-Target conversion out of
Irker.handle entirely, so it can focus more strongly on dispatch and
connection management.

irkerd: Add InvalidRequest and use it to flatten Irker.handle()

The old implementation had several instances of logic like this:

  if exception_condition:
      self.logerr("invalid request")
  else:
      # continue_processing

This increases nesting after each round of exception checking, and
makes the logic of the whole function harder to follow.  This commit
replaces that logic with:

  try:
      if exception_condition:
          raise InvalidRequest("invalid request")
      # continue peocessing
  except InvalidRequest, e:
      self.logerr(str(e))

Because the guts of the handle() function are already inside a
try/except block, we can add our except clause to the existing block,
and now exception checks don't increase nesting at all.

The exception to this global try/except block is the 'URL has
unexpected type' error, where we do want a local try/except block
inside the channel loop.  That way we get both errors about invalid
URLs and continue to attempt valid URLs.  This matches the existing
logic for this check, but conflicts with the current target.valid
check (which doesn't log an error and does stop processing of further
channels).

irkerd: Store less state in IRCServerConnection.connect()

We will never need the connection-time port, server_address, username,
ircname, or password again, so don't store them.  We *do* need server
and real_server_name for Event handling, so keep them around.

Version bump for 2.6.

Add a check for expired connections that are still running

Don't loop forever after failing to connect to IRC

AI0867 gets more credit.

Documentation improvements.

Version bump for 2.5 release.

Fix a deadlock

My typo fix was typoed.  Clearly I need more sleep.

Typo fix.

Documentation tweak.

Stupid markup fix.

Version bump for release 2.4.

Only convert to unicode when needed

Otherwise, it would crash if any unicode string was already
present.

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

Supply a missing parameter.

Cleaner exit code.

We don't want to return an error status in immediate mode.

fix crash after introduction of immediate mode

this fixes the following backtrace:

Exception happened during processing of request from ('127.0.0.1', 41192)
Traceback (most recent call last):
  File /usr/lib/python2.7/SocketServer.py, line 295, in _handle_request_noblock
    self.process_request(request, client_address)
  File /usr/lib/python2.7/SocketServer.py, line 321, in process_request
    self.finish_request(request, client_address)
  File /usr/lib/python2.7/SocketServer.py, line 334, in finish_request
    self.RequestHandlerClass(request, client_address, self)
  File /usr/lib/python2.7/SocketServer.py, line 649, in __init__
    self.handle()
  File ./irkerd, line 820, in handle
    irker.handle(line.strip())
  File ./irkerd, line 786, in handle
    self.servers[target.server()].dispatch(target.channel, message, target.key, quit_after=quit_after)
  File ./irkerd, line 641, in dispatch
    eligibles[0].enqueue(channel, message, key)
TypeError: enqueue() takes exactly 5 arguments (4 given)

when running: ./irk irker-test test

add a more helpful hook example: in post-receive

the previous suggestion seemed to be about the update hook, but it's
usually bad practice to setup notification hooks there, because if
they fail, the push fails.

parsing the git documentation to find exactly the incantation is not
exactly trivial either...

Another shipper metadata change.

Version bump for 2.3 release.

Reinstate a simpler irk that only works with irkerd running.

Clean up and document immediate mode.

Immediate mode work.

Remove unecessary elaboration.

An until string is not the answer.

Turning off DEAF is mecessary for this technique...

...but, alas, the server doesn't echo pings back to their sources.

I think this would work to terminate immediaate mode...

...if we could see message traffic!

Refactoring step: don't uncomditionally spawn a thread...

...immediate mode doesn't need it.

Note a deficiency

Half-working immediate mode.

We can now send messages with -i but we don't ger clean termination
afterwards yet.

NEWS typo fix.

Version bump for 2.2 release.

Fix up some shipper metadata.

Required magic to kill the spawned instance.

Show traceback on higher debug levels

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

Fix some documentation typos

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

An attempt at making irk clean up after itself.

Perplexingly, the terminate() method call doesn't.

Fix unicode processing

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

Remove some irclib cruft

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

Version bump for release 2.1.

Undo some damage from an overambitious patch.

More documentation polishing.

weechat doesn't do ANSI color, only mIRC.

News update.

Explain why relaying through this is a good idea!

Documentation polishing.

Cosmetic fix to Makefile.

Be more flexible about where irker can be installed.

Clean up the make file and use a more portable method of making tarballs.

Merge commit 'refs/merge-requests/26' of git://gitorious.org/irker/irker into merge-requests/26

Adapt for new shipper conventions.

Speed up parsing the Git commit author and subject

This uses the same idiom as a few lines lower to split the
formatted message into the three fields.

Version bump for 2.0.

Improved .gitignore based om an idea bu dak180.

Slightly more elegant fix, no need for new connectfail status.

Meatball-surgery fix for invalid-name bug.

00:07:27   AI0867 | esr: it reproduces on git head                   │
00:07:42   AI0867 | ./irk irc://chat.freendoe.net/foo                │
00:07:45   AI0867 | that's sufficient                                │
00:08:01      esr | OK, please email me a description of how to      │
                  | reproduce, I'll fix it.                          │
00:08:39      esr | Oh.  Is therec anything special about that       │
                  | channel?                                         │
00:08:42   AI0867 | no                                               │
00:08:48   AI0867 | any incorrect servername will do                 │
00:08:56   AI0867 | it then attempts to quit the server              │
00:09:01   AI0867 | and tries to send a QUIT                         │
00:09:06   AI0867 | which throws an exception                        │
00:09:10   AI0867 | so it tries to quit the server                   │

Add a todo note.

Remove an irclib remnant.

More debug consolidation.

More uniform logging.

Fix the spin loop not to be O(n**2).

Smoke test passes.

Remove unnecessary mutex lock.

Eliminate the dependency on irclib.

Repairing the secret-chanel patch.  Smoke test works.

Preparing for release.

Prevent a unicode error on UTF-8 in commit metadata.

Having utf-8 in the commit metada (e.g author, commit message, file
names) triggered a decoding error:

"UnicodeDecodeError: 'ascii' codec can't decode byte 0xd7 in position 42: ordinal not in range(128)"

This is because the __unicode__ method of the Commit class, doesn't
actually return an object of type 'unicode' although it should.

Update the news.

Merge commit 'refs/merge-requests/22' of git://gitorious.org/irker/irker into merge-requests/22

Add support for keyed channels. Supports both channel?secret and channel?key=secret.

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

Allow irkerhook to be used as a mecurial changegroup hook

Noise reduction.

Polishing irk.

Upgrade the test tool.

Improve an error message.

Shut up, pylint!

Dcumentation update.

Add the 'email' variable to the irkerhook manpage

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

Implement email as a delivery method

This delivery method exists for the benefit of sourceforge,
which does not allow hooks to access the network except through email.

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

Version bumpp for 1.20 release.

Compatibility with python 2.4 (if simplejson is present)

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

Increase anti-flood-delay from 0.5 to 1 second.

With a delay of 0.5 seconds, freenode starts dropping messages
bunched in large groups after about 18 of them.

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

Version bump for 1.9 release.

Wrote irkerd usage

- invalid options should print usage

irkerd -V shouldn't take an argument

Version bump for 1.18 release.

Do not write to logfile if None

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

Python 2.5 compatibility

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

Document new options properly.

Append mode is required if we're loogging to disk.

Prevent actually sending an empty privmsg.

Added -p option.