фикс noserver.pl

[elite]

Предлагаю небольшой фикс noserver.pl

Код:

# Sql выборки всех авторизованных клиентов, которым не запрещен доступ и у которых подключена услуга с тегом `inet`

my $sql_get_auth_usr = <<SQL;
    SELECT u.id, u.balance, u.name, u.state, a.auth_start, a.ip, a.properties
        FROM (
        SELECT ip, properties, start AS auth_start FROM auth_now
                UNION ALL
        SELECT INET_NTOA(i.ip) AS ip, '' as properties, UNIX_TIMESTAMP() AS auth_start
            FROM users u JOIN ip_pool i ON i.uid=u.id WHERE u.lstate=1.
    ) a.
        JOIN ip_pool i ON INET_ATON(a.ip)=i.ip
        JOIN users u ON i.uid=u.id
    WHERE TRUE
SQL

и в функции load_usr_info
изменить

Код:

my $db = Db->sql( $sql_get_auth_usr );

на

Код:

my $db = Db->sql( $sql_get_auth_usr.' GROUP BY ip' );

Смысл: если у клиента стоит состояние "всегда онлайн", а он авторизуется чем-либо (dhcp, pppoe и т.п.), то noserver "видит" его как 2 разных ip. Данным фиксом убираем эту проблему

[Efendy]

Этот sql будет некорректным хотя и будет выполнен mysql - нельзя выбирать неагрегированные поля, не участвующие в группировке. Со временем они это пофиксят и будут проблемы.

[elite]

Тогда как правильно решить задачу?

[Efendy]

Смотря какую запись считать приоритетной

[elite]

Смотря какую запись считать приоритетной

приоритетной считать из auth_now

[Efendy]

Слегка протестил это:

Код:

    SELECT u.id, u.balance, u.name, u.state, a.auth_start, a.ip, a.properties
        FROM (
        SELECT INET_NTOA(i.ip) AS ip, '' as properties, UNIX_TIMESTAMP() AS auth_start
            FROM users u JOIN ip_pool i ON i.uid=u.id WHERE u.lstate=1
                AND NOT EXISTS (SELECT 1 FROM auth_now WHERE INET_ATON(ip)=i.ip)
                UNION ALL
        SELECT ip, properties, start AS auth_start FROM auth_now
    ) a 
        JOIN ip_pool i ON INET_ATON(a.ip)=i.ip
        JOIN users u ON i.uid=u.id
    WHERE TRUE