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

Проверка на баны #43

Open
venticello opened this issue Nov 13, 2014 · 5 comments
Open

Проверка на баны #43

venticello opened this issue Nov 13, 2014 · 5 comments

Comments

@venticello
Copy link
Contributor

Текущая админка насилует базу при каждом заходе нового пользователя на сайт, даже если он не авторизован, посылает запросы в базу. По моему это надо делать только для авторизованных пользователей иначе неадекватно растет нагрузка.

@psnet
Copy link
Contributor

psnet commented Nov 13, 2014

По моему это надо делать только для авторизованных пользователей иначе неадекватно растет нагрузка.

баны могут быть и по диапазону айпи, без сущности пользователя.

@venticello
Copy link
Contributor Author

Для большинства случаев установки админки, банов будет минимум только по IP. Можно выбирать сразу все баны и сравнивать значение из кэша, и каждый раз в базу не лезть за новым гостем на сайте.

@psnet
Copy link
Contributor

psnet commented Nov 14, 2014

Я как раз кеширование специально не делал для банов из расчета, что правил может быть со временем много.

@venticello
Copy link
Contributor Author

Из-за чего это стал писать, у меня в настройках мускула установлена настройка логировать запросы в базу без индексов, и у меня только вот этот запрос за 2 дня 7кб логов наплодил. Хотя я посмотрел индексы есть.
Не знаю как правильно тут реализовывать для универсальности, но когда банов нет, вообще печалит, что он базу постоянно опрашивает.

@psnet
Copy link
Contributor

psnet commented Nov 14, 2014

Хотя я посмотрел индексы есть.

там такой хитрый запрос и вполне может не использовать индексы, связано с тем что баны имеют множество опций (сущность/айпи/диапазон, тип блокировки, время и что-то ещё). И чтобы выполнять эту проверку за один запрос пришлось сделать его не самым простым.

Но согласен, что подлежит пересмотру.

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