Correctly filter target list for slavesHEADmaster

The list of slaves enabled for a target is a space-separated string. A
slave name can be a substring of another slave name, so matching it
using /\b${slave}\b/ is not accurate.

This is an issue when we have two slaves named "abc" and "abc-def"
and a target defined only for "abc". Currently, both slaves receive a
config including the target.

1 files changed, 1 insertions, 1 deletions

diff --git a/lib/Smokeping/Master.pm b/lib/Smokeping/Master.pm
index 869d923..49cd721 100644
--- a/lib/Smokeping/Master.pm
+++ b/lib/Smokeping/Master.pm
@@ -43,7 +43,7 @@ sub get_targets {
         # master
         next if $key eq 'host' and $trg->{$key} eq 'DYNAMIC';
         next if $key eq 'host' and $trg->{$key} =~ m|^/|; # skip multi targets
-        next if $key eq 'host' and not ( defined $trg->{slaves} and $trg->{slaves} =~ /\b${slave}\b/);
+        next if $key eq 'host' and not ( defined $trg->{slaves} and grep { $_ eq $slave } split /\s+/, $trg->{slaves});
         if (ref $trg->{$key} eq 'HASH'){
             $return{$key} = get_targets ($trg->{$key},$slave);
             $ok = 1 if defined $return{$key};