Delta Between Two Patch Sets: modules/fail2ban/templates/jail.erb
Issue 29364214:
Issue 2487 - Introduce fail2ban module (Closed)
Left Patch Set:
Created Nov. 28, 2016, 3:59 p.m.
Right Patch Set: For comment 22 and 23
Created Dec. 2, 2016, 2:22 p.m.
| Left: | ||
| Right: |
Use n/p to move between diff chunks;
N/P to move between comments.
« no previous file with change/comment
|
« modules/fail2ban/templates/filter.erb ('k')
|
modules/private-stub/hiera/base.yaml » ('j')
|
no next file with change/comment »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Hide Comments ('s')
| LEFT | RIGHT |
|---|---|
| 1 <% @jail_config.each do |name, config| -%> | 1 <% @jails.each do |name, config| -%> |
| 2 <% if !config['logpath'].empty? -%> | 2 <% if !config['logpath'].empty? -%> |
| 3 [<%= name %>] | 3 [<%= name %>] |
| 4 <% if config['filter'].empty? -%> | |
| 4 filter = <%= name %> | 5 filter = <%= name %> |
| 6 <% end -%> | |
| 5 <% configuration = scope.function_merge(@jail_default, config) -%> | 7 <% configuration = scope.function_merge(@jail_default, config) -%> |
| 6 <% configuration.each do |key, value| -%> | 8 <% configuration.each do |key, value| -%> |
| 7 <%= key %> = <%= value %> | 9 <%= key %> = <%= value %> |
| 8 <% end -%> | 10 <% end %> |
| 11 <% else -%> | |
| 12 <% raise Puppet::Error, "Log path missing from configuration" -%> | |
| 9 <% end -%> | 13 <% end -%> |
|
f.nicolaisen
2016/11/29 00:38:55
I think if you remove the minus (-) above here you
| |
| 10 <% end -%> | 14 <% end -%> |
|
f.nicolaisen
2016/11/29 00:38:55
Missing empty line for end of file.
| |
| 15 | |
| LEFT | RIGHT |
« modules/fail2ban/templates/filter.erb ('k')
|
modules/private-stub/hiera/base.yaml » ('j')
|
Toggle Intra-line Diffs ('i')
|
Expand Comments ('e')
|
Collapse Comments ('c')
|
Toggle Comments ('s')
