Actions
Bug #11434
open
SquidGuard over 1.16.18_11
Status:
Feedback
Priority:
Normal
Assignee:
-
Category:
squidguard
Target version:
-
Start date:
02/17/2021
Due date:
% Done:
0%
Estimated time:
Plus Target Version:
Affected Version:
Affected Plus Version:
Affected Architecture:
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
| screen2.PNG (17.2 KB) screen2.PNG | |||
| screen1.PNG (30.1 KB) screen1.PNG | |||
| screen3.PNG (18.3 KB) screen3.PNG |
Updated by 
Updated by Eduardo Silva about 2 years ago
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.
Updated by Eduardo Silva about 2 years ago
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]);
}
Updated by Renato Botelho about 2 years ago
- Status changed from New to Feedback
- Assignee set to Renato Botelho
Fix pushed to version 1.16.18_15. Thank you!
Actions