Bug #11434
open
SquidGuard over 1.16.18_11
Added by Robson Ferreira over 1 year ago.
Updated 3 months ago.
Description
Hi, first problem thatI found is when we are using ldapusersearch on groups acl.
In older version when I insert ldapusersearch it was show on filter log. But now this option is not applicable, and that is the reason that filter not happens.
Ill send pictures .
Files
Hi, i have same problem. i try change Client (source) with ip, domain, etc.. and config file is written with correct parameter, but with ldapusersearch, the parameter is not written in config file.
I found the problem in /usr/local/pkg/squidguard_configurator.inc
Line: 903
# separate IP, domains, usernames
if (strpos(trim($src[F_SOURCE]), 'ldapusersearch') === false) {
$tsrc = explode(" ", trim($src[F_SOURCE]));
foreach($tsrc as $sr) {
$sr = trim($sr);
if (empty($sr)) {
continue;
}
if (is_ipaddr_valid($sr)) {
$sg_tag->items[] = "ip $sr";
} elseif (is_domain_valid($sr)) {
$sg_tag->items[] = "domain $sr";
} elseif (is_username($sr)) {
$sg_tag->items[] = "user " . str_replace("'", "", $sr);
} else {
$sg_tag->items[] = trim($src[F_SOURCE]);
}
}
}
i change with:
# separate IP, domains, usernames
if (strpos(trim($src[F_SOURCE]), 'ldapusersearch') === false) {
$tsrc = explode(" ", trim($src[F_SOURCE]));
foreach($tsrc as $sr) {
$sr = trim($sr);
if (empty($sr)) {
continue;
}
if (is_ipaddr_valid($sr)) {
$sg_tag->items[] = "ip $sr";
} elseif (is_domain_valid($sr)) {
$sg_tag->items[] = "domain $sr";
} elseif (is_username($sr)) {
$sg_tag->items[] = "user " . str_replace("'", "", $sr);
}
}
} else {
$sg_tag->items[] = trim($src[F_SOURCE]);
}
- Status changed from New to Feedback
- Assignee set to Renato Botelho
Fix pushed to version 1.16.18_15. Thank you!
- Assignee deleted (
Renato Botelho)
Also available in: Atom
PDF