Put some last missing pieces together.

* Add some utility transport functions in circuitbuild.[ch] so that we
  can use them from pt.c.
* Make the accounting system consider traffic coming from proxies.
* Make sure that we only fetch bridge descriptors when all the
  transports are configured.

Create the pluggable_transports.[ch] source files responsible for talking the 180 talk.

Make some utility functions.

* Create a function that will get input from a stream, so that we can
  communicate with the managed proxy.
* Hackish change to tor_spawn_background() so that we can specify an
  environ for our spawn.

Add support for managed {Client,Server}TransportPlugin parsing.

Refactor the interfaces of transport/proxy lookup fns

Returning a tristate is needless here; we can just use the yielded
transport/proxy_type field to tell whether there's a proxy, and have
the return indicate success/failure.

Also, store the proxy_type in the or_connection_t rather than letting
it get out of sync if a configuration reload happens between launching
the or_connection and deciding what to say with it.

Future-proof and user-proof parse_bridge_line

Simplify parse_client_transport_line

Style and grammar tweaks on 2841 branch

Small tweaks to 2841 code

    - const-ify some transport_t pointers
    - Remove a vestigial argument to parse_bridge_line
    - Make it compile without warnings on my laptop with
      --enable-gcc-warnings

Trivial code tweaks and documentation updates.

Revised how we handle ClientTransportPlugin and Bridge lines.

Multiple Bridge lines can point to the same one ClientTransportPlugin
line, and we can have multiple ClientTransportPlugin lines in our
configuration file that don't match with a bridge. We also issue a
warning when we have a Bridge line with a pluggable transport but we
can't match it to a ClientTransportPlugin line.

Various small tweaks around config.c and or.h

Tweaked connection{.c,.h,_or.c} based on nick's comments.

* Tweaked doxygen comments.
* Changed returns of get_proxy_addrport().
* Ran make check-spaces.
* Various small code tweaks.

Fixes on circuitbuild.[ch] based on nick's comments.

* Renamed transport_info_t to transport_t.
* Introduced transport_get_by_name().
* Killed match_bridges_with_transports().
  We currently *don't* detect whether any bridges miss their transports,
  of if any transports miss their bridges.
* Various code and aesthetic tweaks and English language changes.

Fixes small bugs.

This commit is an attempt to beautify the previous commit.

It creates some helper functions that return the proxy type, proxy addr/port, etc.

Our warning now is much more specific, mentioning proxy type/addr/port.

Not included in the previous commit, because the implementation is
ugly; I see no other way of doing this though.

We now warn the user if a proxy server is not up when we try to connect with it.

Various trivial changes.

* Improved function documentation.
* Renamed find_bridge_transport_by_addrport() to
  find_transport_by_bridge_addrport().
* Sanitized log severities we use.
* Ran check-spaces.

We can now connect using transports as well!

We can now match our transports with our bridges.

ClientTransportPlugin parsing done.

fix typo in changes/coverity_master spotted by rransom

Move errant doc/nodefamily_routerset into 0.2.3.1-alpha ChangeLog. Fix bug 3373

Upgrade to the latest version of tinytest

Prefer tt_assert in unit tests, not tor_assert

tweak coverity_master changes file

Fix a memleak in nodelist_assert_ok()

Remove a dead variable in the heartbeat code

Fix a minor coverity-found bug in tests

Update the uptime string to include an hours indicator

Before, it wasn't really obvious if one meant hours:minutes or
minutes:seconds etc.

Fix crash bug (3361) when setting off-flavor networkstatus

When we set a networkstatus in the non-preferred flavor, we'd check
the time in the current_consensus.  But that might have been NULL,
which could produce a crash as seen in bug 3361.

Merge remote-tracking branch 'asn2/bug3336'

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge branch 'bug3306_nm_squashed' into maint-0.2.2

Detect insanely large circuit build state; don't give its length to rand_int

Check maximum properly in crypto_rand_int()

George Kadianakis notes that if you give crypto_rand_int() a value
above INT_MAX, it can return a negative number, which is not what
the documentation would imply.

The simple solution is to assert that the input is in [1,INT_MAX+1].
If in the future we need a random-value function that can return
values up to UINT_MAX, we can add one.

Fixes bug 3306; bugfix on 0.2.2pre14.

Add the heartbeat domain in log.c:domain_list[]
so that parse_log_domain() doesn't fail.

Merge branch 'maint-0.2.2'

Merge branch 'maint-0.2.1' into maint-0.2.2

move to the june 1 2011 maxmind geoip db

port-forward the 0.2.2.28-beta changelog stanza

put the changelog stanzas in time order

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge branch 'bug3318c' into maint-0.2.2

Reject 128-byte keys that are not 1024-bit

When we added the check for key size, we required that the keys be
128 bytes.  But RSA_size (which defers to BN_num_bytes) will return
128 for keys of length 1017..1024.  This patch adds a new
crypto_pk_num_bits() that returns the actual number of significant
bits in the modulus, and uses that to enforce key sizes.

Also, credit the original bug3318 in the changes file.

Merge remote-tracking branch 'origin/maint-0.2.2'

Use an autobool for UseBridges_

Merge remote-tracking branch 'rransom/bug2748-v2' into maint-0.2.2

Merge remote-tracking branch 'origin/maint-0.2.2'

Changes file for bug2355.

Update man page for new UseBridges tristate behaviour.

Add an "auto" option to UseBridges

UseBridges 1 now means "connect only to bridges; if you know no
bridges, don't make connections."  UseBridges auto means "Use bridges
if they are known, and we have no EntryNodes set, and we aren't a
server."  UseBridges 0 means "don't use bridges."

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/rendclient.c

Merge remote-tracking branch 'rransom-tor/bug3309' into maint-0.2.2

Merge remote-tracking branch 'origin/maint-0.2.2'

This merge was a bit nontrivial, since I had to write a new
node_is_a_configured_bridge to parallel router_is_a_configured_bridge.

Conflicts:
src/or/circuitbuild.c

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/dirserv.c

Merge remote-tracking branch 'arma/bug3321' into maint-0.2.2

Fix unit test failure in dir/formats

options->DirPort is 0 in the unit tests, so
router_get_advertised_dir_port() would return 0 so we wouldn't pick a
dirport. This isn't what we want for the unit tests. Fixes bug
introduced in 95ac3ea5946.

Add info-level log messages during HS-client-state purge

I hope these will never be useful, but having them and not needing them is
better than needing them and not having them.

Refactor HS client state-clearing code into a separate function

Clear last_hid_serv_requests on SIGNAL NEWNYM

Fixes bug #3309.

Make last_hid_serv_requests functions less fragile

Previously, Tor would dereference a NULL pointer and crash if
lookup_last_hid_serv_request were called before the first call to
directory_clean_last_hid_serv_requests.  As far as I can tell, that's
currently impossible, but I want that undocumented invariant to go away
in case I^Wwe break it someday.

Unbreak the build

Merge remote-tracking branch 'origin/maint-0.2.2'

Fix compile error in procmon.c

An elusive compile-error (MingW-gcc v4.50 on Win_XP); a missing
comma (!) and a typo ('err_msg' at line 277 changed to 'errmsg').
Aso changed the format for 'err_code' at line 293 into a "%ld" to suppress
a warning. How did this go unnoticed for ~1 month? Btw. This is my 1st ever
'git commit', so it better work.

Merge remote-tracking branch 'origin/maint-0.2.2'

Report wrong key sizes correctly

When we introduced NEED_KEY_1024 in routerparse.c back in
0.2.0.1-alpha, I forgot to add a *8 when logging the length of a
bad-length key.

Bugfix for 3318 on 0.2.0.1-alpha.

fix a bridge edge case similar to 2511

If you had configured a bridge but then switched to a different bridge
via the controller, you would still be willing to use the old one.

stop asserting at boot

The patch for 3228 made us try to run init_keys() before we had loaded
our state file, resulting in an assert inside init_keys. We had moved
it too early in the function.

Now it's later in the function, but still above the accounting calls.

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge branch 'bug3216_v2' into maint-0.2.2

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge branch 'bug3289' into maint-0.2.2

Warn when two hs use the same directory

This simple implementation has a few issues, but it should do for
0.2.2.x. We will want to revisit this later and make it smarter.

Merge remote-tracking branch 'origin/maint-0.2.2'

The conflicts were mainly caused by the routerinfo->node transition.

Conflicts:
src/or/circuitbuild.c
src/or/command.c
src/or/connection_edge.c
src/or/directory.c
src/or/dirserv.c
src/or/relay.c
src/or/rendservice.c
src/or/routerlist.c

minor cleanups while reviewing 3216

whitespace fixes

Merge branch 'bug3045' into maint-0.2.2

Conflicts:
src/or/circuitbuild.c

Improve comments and defensive programming for 3045

The comment fixes are trivial.  The defensive programming trick is to
tolerate receiving NULL inputs on the describe functions. That should
never actually happen, but it seems like the likeliest mistake for us
to make in the future.

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/common/compat.c
src/or/main.c

Merge remote-tracking branch 'public/bug3270' into maint-0.2.2

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge branch 'bug3228_squashed' into maint-0.2.2

Reinit keys at the start of options_act().

Previously we did this nearer to the end (in the old_options &&
transition_affects_workers() block).  But other stuff cares about
keys being consistent with options... particularly anything which
tries to access a key, which can die in assert_identity_keys_ok().

Fixes bug 3228; bugfix on 0.2.2.18-alpha.

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/circuituse.c

Merge branch 'bug1297a' into maint-0.2.2

Use the normal four-hop CBT for client intro circuits

Fixes another part of bug 1297.

Set timestamp_dirty on HS circuits as circuit_expire_building requires

Fixes part of #1297; bugfix on 48e0228f1e031a709c1deb149c7dfd187c3609cf,
when circuit_expire_building was changed to assume that timestamp_dirty
was set when a circuit changed purpose to _C_REND_READY.  (It wasn't.)

Merge branch 'maint-0.2.2'

answer an XXX nickm asked in aa950e6c4

Merge remote-tracking branch 'origin/maint-0.2.2'

Merge remote-tracking branch 'origin/maint-0.2.1' into maint-0.2.2

Fix typo in changes/bug2574. Thanks, rransom

Merge remote-tracking branch 'origin/maint-0.2.2'

Conflicts:
src/or/eventdns.c

Merge remote-tracking branch 'origin/maint-0.2.1' into maint-0.2.2

Merge branch 'bug2574' into maint-0.2.1

Fix a -Wunused-but-set-variable instance in master

Merge remote-tracking branch 'origin/maint-0.2.2'

Don't try to build descriptors when router_get_advertised_or_port()==0

The previous attempt was incomplete: it told us not to publish a
descriptor, but didn't stop us from generating one.  Now we treat an
absent OR port the same as not knowing our address.  (This means
that when we _do_ get an OR port, we need to mark the descriptor
dirty.)

More attempt to fix bug3216.