Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

nick='_shareban_' #200

Closed
pavel-pimenov opened this issue Mar 8, 2023 · 3 comments
Closed

nick='_shareban_' #200

pavel-pimenov opened this issue Mar 8, 2023 · 3 comments

Comments

@pavel-pimenov
Copy link
Contributor

В базу идет запрос вида

select share_size, note_usr, note_op, reason, nick_op, last_hit, date_limit, 
date_start, range_to, range_fr, host, ban_type, nick, ip
from banlist 
where ((nick='_shareban_' AND share_size = '1234554321')) 
and ((`date_limit` >= 1678265615) or (`date_limit` is null) or (`date_limit` = 0))
order by `date_limit` desc limit 1

я предполагаю, что обычно такой бан не используют.
предлагаю проверить, что таких записей в таблице нет и более не слать такой запрос.

@RoLex
Copy link
Member

RoLex commented Mar 8, 2023

не часто используют, согласен, но и не реже чем hostban например. запрос довольно тяжелый для сервера, тоже согласен, в особенности последняя часть сортировки и возвращения самого свежего бана. но такой функционал имеется в верлике - бан по шаре. во первых я увидел что нулевая шара тоже проверяется - минус пару запросов в корзину, но если честно не помню, нарочно так было сделано или нет. во вторых ник проиндексирован, и если индекса shareban не существует, мускул сразу вернет ноль. далее date_limit который тоже проиндексирован. нулевую шару исправлю, а остальное требуется для функционала.

@RoLex RoLex closed this as completed in e8b89c6 Mar 8, 2023
@pavel-pimenov
Copy link
Contributor Author

ну а если проверить при старте один раз, что таблица banlist вообще пустая и не слать запросы к ней (пока не будет insert)? у KCAHDER именно так и есть, а хаб крупный.

@RoLex
Copy link
Member

RoLex commented Mar 8, 2023

я не видел хаба без банов, у ксандера в том числе. я конечно тебе очень сильно благодарен за твои усилия, и многие из них рельно полезны, но в данном случае уже перебор :-D поправь пожалуйста вот этот баг в флае > pavel-pimenov/flylinkdc-r6xx#50 < не можем уже смотреть на повторяющиеся описания, половина верликов засраны ими

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants