Правила форума Гаранты форума
Размещение рекламы AMX-X компилятор

Здравствуйте, гость Вход | Регистрация

Наши новости:

14-дек
Добавлена поддержка utf8mb4
24-апр
Telegram группа
10-апр
Обновление PHP
11-апр
Раздача читов

История благодарностей участнику -=Денис=- ::: Спасибо сказали: 6
Дата поста: В теме: За сообщение: Спасибо сказали:
20.8.2013, 19:41 Новый Amx Monitoring 1.1.6
Цитата(kuznets92 @ 17.8.2013, 15:44) *
Дизайн не играет никакой роли, это не главное. По мне он вполне даже.
Вот это выскочило после установки:
[attachment=20828:monitoring.jpg][attachment=20826:monitoring_2.jpg]
Подобные ошибки при переходе на любую страницу.

Больше всего не понравился код, нет никакого стиля, нет отступов. Например, я придерживаюсь вот этого стиля - http://www.zendframework.com/manual/1.12/r...ding-style.html


Мне кажется, когда в коде подобное:
Код
echo "<center><br><br><br><br><br><br><br><br><br><br><b><font size='3'>".$locale['reg029']."</font></b></center><br><br><br><br><br><br><br><br><br><br><br><br>";

То дизайн очень трудно переделать под свои нужны. Разве переписать половину кода мониторинга.

Почему cron можно выполнять прямо из браузера? Сделайте, например, чтобы он выполнялся только из командной строки.
http://free.amxservers.ru/cron.php
[attachment=20829:monitorin_cron.jpg]

Нет проверок на исполнение скриптов. Я могу исполнить любой скрипт мониторинга. Например http://free.amxservers.ru/include/rus_name_fix.php


Дизайн играет очень большую роль, для прогеров почти не играет, но так других то больше! Поэтому большинство людей смотрят на дизайн (проверенно). Некоторые покупали платную версию обосновывая тем что им дизайн на демке не нравится и все.
По поводу кода ответил сообщением выше,
а по поводу крона, сделал без защиты специально, так как многие его запускают через wget из командной строки и не хотят запускать например с параметром file или у многих только через wget разрешено. Просто я уже столько раз с этим сталкивался, что проще так сделать чтобы поняли и то до сих пор спрашивают как и что.
Насчет файлов, то ты далеко не все откроешь так напрямую, админские защищенные, а тот что ты дал он остался из-за того что где то может выскочить ошибка, он переводил название сервера в другую кодировку, щас все в UTF8

Цитата(kuznets92 @ 17.8.2013, 16:09) *
Защиты от CSRF, как я понял, тоже нет.
Стоит мне встроить куда нибудь подобную картинку:
Код
<img src="http://free.amxservers.ru/admin/index.php?id=obpost&new_serv_gl&serv_all_sp&serv=5&del" />

и у админа будет удален сервер с id=5

А ты сам то пробывал ? все админские скрипты не открыть как ты написал

Цитата(kuznets92 @ 17.8.2013, 16:16) *
Код
    if (isset($_GET['id']))
  {
    if ($_GET['id'] == "add")
    {
      require "register.php";
    }
    elseif ($_GET['id'] == "all_servers")
    {
      require "all_servers.php"
    }else{
        error_404();    
    }


Не лучше ли будет сделать
Код
    switch($_GET['id']) {
        case 'add':
            require "register.php";
            break;
            
        case 'all_servers':
            require "all_servers.php";
            break;
            
        default:    
            error_404();
            break;
    }


switch case работает быстрее if elseif
да и код куда более читаем.


Я написал выше, что кодов не занимался
 Gani97Kz
20.8.2013, 19:31 Новый Amx Monitoring 1.1.6
Цитата(onotole @ 17.8.2013, 11:09) *
Код: 

function isEmail($email)
            {
                return(preg_match("/^[-_.[:alnum:]]+@((([[:alnum:]]|[[:alnum:]][[:alnum:]-]*[[:alnum:]])\.)+(ad|ae|aero|af|ag|ai|al|am|an|ao|aq|ar|arpa|as|at|au|aw|az|ba|bb|bd|be|b
f|bg|bh|bi|biz|bj|bm|bn|bo|br|bs|bt|bv|bw|by|bz|ca|cc|cd|cf|cg|ch|ci|ck|cl|cm|cn
|
co|com|coop|cr|cs|cu|cv|cx|cy|cz|de|dj|dk|dm|do|dz|ec|edu|ee|eg|eh|er|es|et|eu|f
i
|fj|fk|fm|fo|fr|ga|gb|gd|ge|gf|gh|gi|gl|gm|gn|gov|gp|gq|gr|gs|gt|gu|gw|gy|hk|hm|
h
n|hr|ht|hu|id|ie|il|in|info|int|io|iq|ir|is|it|jm|jo|jp|ke|kg|kh|ki|km|kn|kp|kr|
k
w|ky|kz|la|lb|lc|li|lk|lr|ls|lt|lu|lv|ly|ma|mc|md|mg|mh|mil|mk|ml|mm|mn|mo|mp|mq
|
mr|ms|mt|mu|museum|mv|mw|mx|my|mz|na|name|nc|ne|net|nf|ng|ni|nl|no|np|nr|nt|nu|n
z
|om|org|pa|pe|pf|pg|ph|pk|pl|pm|pn|pr|pro|ps|pt|pw|py|qa|re|ro|ru|rw|sa|sb|sc|sd
|
se|sg|sh|si|sj|sk|sl|sm|sn|so|sr|st|su|sv|sy|sz|tc|td|tf|tg|th|tj|tk|tm|tn|to|tp
|
tr|tt|tv|tw|tz|ua|ug|uk|um|us|uy|uz|va|vc|ve|vg|vi|vn|vu|wf|ws|ye|yt|yu|za|zm|zw
)$|(([0-9][0-9]?|[0-1][0-9][0-9]|[2][0-4][0-9]|[2][5][0-5])\.){3}([0-9][0-9]?|[0-1][0-9][0-9]|[2][0-4][0-9]|[2][5][0-5]))$/i"
                        ,$email));
            }


Посмеялся =)

Лучше было заменить на
Код: 

function isEmail($email)
            {
                return filter_var($email, VILTER_VALIDATE_EMAIL);
            }


Код: 
echo "<html><head><meta http-equiv='Refresh' content='300; URL=/rcon.php'></head><body>";

Не проще ли header('Location: /rcon.php');

Дальше не стал читать. Код из далекого 2005 года. Всё написано криво, невалидно и ужасно



Извини, я не стал полностью перерабатывать бесплатную версию, я ее начал писать когда осваивал php. Щас просто реконструировал функции и т.д, чтобы работало. На код я сильно не смотрел, так как времени делать не было. Зато работает )
 Gani97Kz
25.1.2013, 19:13 Куплю скрипт мониторинга
Цитата(Bindra @ 24.1.2013, 5:01) *
http://amxservers.ru/ - хороший мониторинг и очень дешево, но похоже они собираются сделать снятие копирайта за деньги(

Я б не делал снятие копирайта за деньги, если бы относились нормально, а то каждый второй снимает просто так. Мне обидно просто
 doz, wwwplaton
25.1.2013, 19:06 Куплю скрипт мониторинга
Я бы все равно поспорил, и доказал бы что от нарутки можно защитить. Просто сделать, чтобы скрипт проверял человеческие повадки, В общем скоро увидите в AMX MONITORING ;-)
 doz, wwwplaton

Powered By Invision Power Board  © 2005-2026 
Counter Strike Support Community