smtp_client: rename self -> client (descriptive)
Merge pull request #727 from msimerson/smtp_client4

smtp_client: rename self -> client (descriptive)
fix logic, finally.
headers: add Delivered-To check
karma: 2 changes:

* apply awards before adding message header (so the Header value has data_post
  awards shown)
* permit karma to rejecting during queue hook
* rename config_data -> cfg (concise, and that's

the convention we're using elsewhere (especially in plugins)
Merge pull request #730 from msimerson/server

rename config_data -> cfg (concise, and that's
Merge pull request #728 from msimerson/headers

headers: add Delivered-To check
Merge pull request #729 from msimerson/karma

karma: 2 changes:
server: used named (vs inline anon) functions

and add aliases for host and port (increase readability)
server: consistent config handling

* added load_smtp_ini() with booleans + callback
* merged apply_defaults into load_smtp_ini
* replaced config loading (3x)
* removed cfg passing

It's not yet a convention in our code, but what I've been using consistently
in the plugins and elsewhere is:

   cfg = a reference to whatever is returned by config.get()
   c = a reference to something within cfg
server: moved defaults obj into load_smtp_ini()
p0f: add a register() and load config there

instead of in the master and child inits

and jshint tidyness
Merge pull request #731 from msimerson/server

server: consistent config handling
Fix bug in helo.checks.js
Merge pull request #733 from msimerson/p0f

p0f: add a register() and load config there
Merge pull request #735 from smfreegard/master

Fix timeout bug in helo.checks.js