search

SB-MaterialAdmin / Web

SourceBans Material Design
64 stars 45 forks source link

Недостатки и баги #103

Open rsgua opened 7 years ago

rsgua commented 7 years ago

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

CrazyHackGUT commented 7 years ago

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

Зачем?

При долгом отключению сервера сохраненные ркон пароли пропадают у сервера.

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

Иногда багается вкладка сервера и там идет код вместо норм списка.

Проблема XAJAX, о ней знаем.

rsgua commented 7 years ago
  1. Затем что это проверка вообще не нужна. Если нет эмейла то человек логинется черз стим
  2. Это бред нужно сделать проверку что если сервер офлайн то не удалялись рконы когда сервер даже обновляется то рконы пропадают а это вообще пару менут. Надоело постоянно рконы вводить.
  3. Этот баг существует с самого начала и уже точно прошло пол года и он не исправлен
rsgua commented 7 years ago

Да и еще очень старый баг когда генерируешь ключ ваучера а он генерируется на меньше количество символов чем нужно. Еще бред требовать ввод эмейла и пароля при его активации

CrazyHackGUT commented 7 years ago

Затем что это проверка вообще не нужна. Если нет эмейла то человек логинется черз стим

Ну так пускай логинится через Steam. Но E-Mail вырезать полностью пока не представляется возможным из-за некоторых мест, где всё именно на нём и завязано.

нужно сделать проверку что если сервер офлайн то не удалялись рконы

Она есть. Но работает через раз из-за особенностей работы протоколов TCP/UDP.

Этот баг существует с самого начала и уже точно прошло пол года и он не исправлен

Потому что нельзя просто взять и исправить. Этот баг уходит далеко в недры библиотеки XAJAX. Чтобы исправить его, надо саму либу переписывать, а конкретно класс xajaxResponse.

Да и еще очень старый баг когда генерируешь ключ ваучера а он генерируется на меньше количество символов чем нужно.

В курсе, связано с браузерами. Один браузер адекватно хандлит событие изменения содержимого инпута, другой нет, и так далее...

Еще бред требовать ввод эмейла и пароля при его активации

Господи, чего Вы так придрались к этому Емейлу и паролю? Я всё понимаю, прошлый век, но нельзя вот так взять и выбросить насовсем E-Mail и пасс.

CrazyHackGUT commented 7 years ago

И вообще, если не просите E-Mail адреса у своих администраторов, тупо вбивайте рандомное мыло, которое подходит под регулярку. Всегда своим админам вбивал что-то вроде adminXXXX@g-44.ru, и похер)

rsgua commented 7 years ago

понимаешь 90% проектов которые используют соресбанс это комерческие... и админки там покупаються. зачем юзеру еще доп поля вводить которые по сути логики и не нужны. и я вообще на твоем месте сделал SB платным и доделал его до ума. у тебя есть аудетория для его покупки. а он стоит уже сколько времени без исправлений.