Merge branch 'master' into 'master'

Fix deprecated variable references

See merge request !3

Fix deprecated variable references

Merge branch 'bugfix/issue-1-unset_outbound_bindaddresses' into 'master'

tor::daemon::relay: set $real_outbound_bindaddresses to [] when $outbound_bindad…

…dresses is not explicitly set.

Closes: #1
See merge request !2

tor::daemon::relay: set $real_outbound_bindaddresses to [] when $outbound_bindaddresses is not explicitly set.

Closes: #1

Merge branch 'improvements' into 'master'

Improvements

These are a bunch of improvements I got on my github profile in the past.

Interested in merging them?

Summary:

* adds tor repo management
* removes a workaround for a fixed bug
* adds tor-arm support
* support for safe logging
* puppet 3 ready templates.

It's merged on top of the current master.

See merge request !1

bug is fixed

fix variable

Merge remote-tracking branch 'github/master'

Conflicts:
templates/torrc.global.erb

Merge pull request #13 from pataquets/add-safe-logging

Add safe_logging parameter.

Merge remote-tracking branch 'shared/master'

Merge remote-tracking branch 'githubmirror/master'

address puppet 3.x deprecation warnings

Merge pull request #10 from pataquets/add-tor-repo-class

Add tor::repo class with apt::source from torproject.org.

Add safe_logging parameter.

Improve tor::repo and add tor::repo::debian classes.

Merge pull request #11 from pataquets/add-arm-class

Add tor::arm class.

Add tor::arm class.

Rename source name parameter.

Add tor::repo class with apt::source from torproject.org .

Merge pull request #1 from irregulator/bug/5241

DirPortFrontPage should serve the 'tor-exit-notice' static page

DirPortFrontPage should serve the 'tor-exit-notice' static page
rather than tor's man page.

Merge pull request #9 from pataquets/remove-unneeded-default-values

Don't add config entries to torrc if value is default

Fix ExitPolicyRejectPrivate not being omitted if set to 1. ( duritong/puppet-tor#5 )

Don't add config entries to torrc if value is default. ( duritong/puppet-tor#5 )

Merge pull request #6 from pataquets/add-PortForwarding

Add PortForwarding to allow UPnP firewall pf configuration.

Add PortForwarding to allow UPnP firewall pf configuration.

Merge pull request #4 from micah/outbound_bindaddresses

Outbound bindaddresses

add upgrade notice to the README to detail the outbound_bindaddress change

Empty template variable for OutboundBindAddress if parameter is an empty array.

Another variable scope fix

Fixing variable scope assingment

Fixing scope lookup at global template

Setting hasstatus and pattern for tor::polipo::debian

Fix missing delimiter

Merge branch 'master' of ssh://labs.riseup.net/shared-tor

Conflicts:
manifests/daemon.pp

fix typo on 2.6

manage a dedicated control port for munin

use the deploy mechanism of the munin module

refactor daemon and main class for better design

By extracting the resources to their own class we can avoid a bug
in puppet that makes it hard to deal with inherited parametrized
classes.

split out defines into their own files

make it working on 2.6 - can't end with a , on 2.6

Linting.

Cleanup: remove now useless variable.

Specify file modes as single-quoted strings.

Puppet style guide "9.6. File Modes" reads:

  In addition, file modes should be specified as single-quoted strings instead
  of bare word numbers.

Move ensure attributes to the first place.

Puppet style guide "9.3. Attribute Ordering" reads:

  If a resource declaration includes an ensure attribute, it should be the first
  attribute specified.

Use the concat module instead of the obsolete concatenated_file define.

... and cleanup the now unused spool_dir.

Give the debian-tor user /bin/false as a shell.

This is the Debian package default, and I've never seen the need to give it
a real shell.

Indentation fixes.

Align all fat arrows in resource’s attribute/value list.

Puppet style guide "9.2. Arrow Alignment" reads: "All of the fat comma arrows
(=>) in a resource’s attribute/value list should be aligned. The arrows should
be placed one space ahead of the longest attribute name."

Quote and enclose in braces variables in strings.

Puppet style guide "8. Quoting" reads: "Double quotes should be used when
variable interpolation is required. [...] All variables should be enclosed in
braces when interpolated in a string."

Use single-quotes for strings that do not contain variables.

Puppet style guide, section "8. Quoting", reads: "All strings that do not
contain variables should be enclosed in single quotes."

Get a lost part of the fix for outbound_bindaddresses back.

The fix was implemented in 8e0dbb0, but apparently I've managed to lose it as
part of one of the numerous merge conflicts I've dealt with when merging
everyone's branch.

Migrate to parameterized classes, drop global variables.

Remove useless comment.

Merge remote-tracking branch 'immerda/master'

Conflicts:
manifests/init.pp
manifests/polipo.pp
manifests/polipo/base.pp
manifests/torsocks.pp

Add support for including arbitrary snippet into torrc.

Merge remote-tracking branch 'shared/master'

Conflicts:
manifests/daemon.pp

Merge remote-tracking branch 'riseup/master' into shared-master

fix template syntax error

add $cookie_authentication, $cookie_auth_file and $cookie_auth_file_group_readable variables

change location of control port in munin graphs, now that
/usr/share/tor/tor-service-defaults-torrc exists and contains:

CookieAuthentication 1
CookieAuthFileGroupReadable 1
CookieAuthFile /var/run/tor/control.authcookie

add README, including information about the transition information regarding the bandwidth parameters

switch bandwidth_rate and bandwidth_burst to be relay_bandwidth_rate and
relay_bandwidth_burst so we can use those variables for their proper tor
configuration variables

Remove trailing whitespace from newly added lines.

Adding support for MapAddress tor option

Adding support for MapAddress tor option

Add support for AutomapHostsOnResolve.

Add support for using bridges.

Add support for TransPort and TransListenAddress.

Add support for DNSPort and DNSListenAddress.

Not managing the top level folder of a hidden service anymore: this can be done in a custom module

Changing data dir permission for a hidden service

Managing the top level folder of a hidden service

The torsocks package does not need the Tor data directory.

add a compact class

mainly to give a way to have the old behavior

Move Polipo configuration files to a dedicated directory.

Factor out Polipo into base and Debian -specific classes.

Decouple torsocks into its own class.

immerda's big decoupling commit (8f1d1c4) is too hard to apply in one shot in
the current state of the shared branch, so let's pick bits of it one after
the other.

new style for 2.7

Merge branch 'bugfix/outbound_bindaddresses-default'

Fix the relay define's outbound_bindaddresses default value setting.

Apparently, we have no guarantee $outbound_bindaddresses's value is set *after*
$listen_addresses is, so (at least on my setup) it ends up being undef,
and then the template parsing fails:

err: Could not retrieve catalog from remote server: Error 400 on SERVER: Failed
to parse template tor/torrc.relay.erb: undefined method `each' for :undef:Symbol
at /etc/puppet/modules/tor/manifests/daemon.pp:117

So, let's set $real_outbound_bindaddresses to $listen_addresses in the define
body, as needed.

enable service on boot

Conflicts:

manifests/init.pp

this file should be deployed after the package have been installed, as it replaces the file from the cronjob

Conflicts:

manifests/init.pp

Don't manage /etc/polipo: it's shipped by the polipo package.

Do not ensure the privoxy package is uninstalled.

this should be done in a seperate module and is not really needed here

Conflicts:

manifests/init.pp

Merge remote-tracking branch 'riseup/master'

fix torc.relay.erb outboundaddress/outboundaddresses confusion

fix duplicate definition of torssocks

outbound_bindaddress should be outbound_bindaddresses and $outbound_bindaddresses  = $listen_addresses

enable different version ensure for torssocks, it doesn't share the same version number as tor and tor-geoipdb, so if you set $tor_ensure_version, you will have an error with torsocks

fix package name for tor-geoipdb

add missing munin plugins that should have been added when the munin.pp was created

add ExitPolicyRejectPrivate option to exit_policy define

add OutboundBindAddress, default set to $listen_address

Remove useless import (closes Redmine #3855).

switch legacy puppet://$server to puppet:///

make sure that the control port's hashed password is not empty

minor formatting standardizing

add tor-geoip package to be installed as its useful for most all installations (#3854)

add variable $tor_ensure_version which allows you to specify a different installed version