Fix action variable in template

[j4m3s]

Fixes #20

[tersmitten]

@j4m3s Do you think:

# The simplest action to take: ban only
action_ = %(banaction)s[name=%(__name__)s, port="%(port)s", protocol="%(protocol)s", chain="%(chain)s"]

# ban & send an e-mail with whois report to the destemail.
action_mw = %(banaction)s[name=%(__name__)s, port="%(port)s", protocol="%(protocol)s", chain="%(chain)s"]
              %(mta)s-whois[name=%(__name__)s, dest="%(destemail)s", protocol="%(protocol)s", chain="%(chain)s"]

# ban & send an e-mail with whois report and relevant log lines
# to the destemail.
action_mwl = %(banaction)s[name=%(__name__)s, port="%(port)s", protocol="%(protocol)s", chain="%(chain)s"]
               %(mta)s-whois-lines[name=%(__name__)s, dest="%(destemail)s", logpath=%(logpath)s, chain="%(chain)s"]

should also be changed?

[j4m3s]

Sorry @tersmitten I'm a bit rusty - which bit should be changed?

[tersmitten]

The expansion of %(banaction)s in jail.local.j2

[j4m3s]

No I think that's correct (as you pasted) - banaction should be expanded as it goes into action_. %(xyz) is just like {$xyz} in php and it needs to be expanded inside action_ otherwise we'll be left with just a "banaction" string.

Sorry if I misunderstood the question :)