Skip to content

Commit

Permalink
Fix bug introduced in #174 (#182)
Browse files Browse the repository at this point in the history
This fixes a bug that was introduced in #174 which doesn't honor the fact if the
bind parameter is nil don't include any bind line. In our case, we put in the
bind keyword in the parameters parameter so this added two bind statements and
made haproxy unhappy.

It looks like this was a copypasta originally as this was included in the prior
code.
  • Loading branch information
ramereth authored and rshade committed Mar 29, 2017
1 parent 8ab9f83 commit f97f29f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion providers/lb.rb
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
end
end
else
listener << "bind #{new_resource.bind}"
listener << "bind #{new_resource.bind}" unless new_resource.bind.nil?
end
listener << "balance #{new_resource.balance}" unless new_resource.balance.nil?
listener << "mode #{new_resource.mode}" unless new_resource.mode.nil?
Expand Down

0 comments on commit f97f29f

Please sign in to comment.