Как правильно отключить часть кода?

Question

[zaart]

Добрый день!

Начал изучать PHP и столкнулся с проблемой. Подскажите, пожалуйста, как отключить прохождение recaptcha, но код при этом в if($responseData->success) { оставался работать?

Ну вот решил я на локалке потестить код и поставил в файле конфига recaptcha = 0

Сам код

if(isset($_POST['submit'])) {

     if(isset($_POST['g-recaptcha-response']) && !empty($_POST['g-recaptcha-response'])) {

        $secret = '*********';

        $verifyResponse = file_get_contents('https://www.google.com/recaptcha/api/siteverify?secret='.$secret.'&response='.$_POST['g-recaptcha-response']);
        $responseData = json_decode($verifyResponse);


             if($responseData->success) {

              // Какой-то код

             } else {
                $errBox = ($lang['form_err_7']);
             }

     } else {
            $errBox = ($lang['form_err_8']);
     }
}

Comments

[Vitsliputsli]

Ну вот решил я на локалке потестить код и поставил в файле конфига recaptcha = 0

Так и сделайте, в чем сложность?

[zaart]

Vitsliputsli, В том, что скрипты рекапчи не загружаются в локаль. Да и просто интересно, как можно реализовать быстрое отключение.

[Антон Р.]

zaart, быстрое отключение так и реализуется 1 или 0 в конфиге.

[zaart]

Антон Р., Я это понимаю. Просто единственное, что мне приходит в голову - это сделать через else две формы. Одна с рекапчей, а вторая без. Но я думаю, что это неправильно и есть способ проще и удобнее. Вот и решил спросить мнения профи.

[Антон Р.]

zaart, сделай проверку на локалку ДО выполнения всего кода и если ip = локальный (не помню точно) то recaptcha = 0, если ip != локалка то recaptcha = 1.

[Vitsliputsli]

Антон Р., это неявное поведение, потом пойди догадайся как приложение будет себя вести исходя из ip или других факторов.

zaart, зачем 2 формы? Одна, но с условием, опираясь на конфиг.

[zaart]

Vitsliputsli, В этом то и проблема, я не могу понять, как правильно написать обработку этого условия.

Можете, пожалуйста, показать на моем куске кода, который я привел выше?

[Vitsliputsli]

zaart, может я не понимаю, в чем проблема... Напишите перед вашим
if(isset($_POST['g-recaptcha-response']) && !empty($_POST['g-recaptcha-response'])) (isset, кстати, здесь не нужен)
что-то вроде:
if ($config->get('recaptcha.enable') === true) {
$config - это объект Конфигурация, если его нет, то нужно создать соответствующий класс, который будет парсить конфиг-файл, и обеспечивать доступ к его свойствам по get-методу.

[zaart]

Vitsliputsli, Сделал как Вы написали, но в итоге код в форме не исполнялся после перевода конфига в false.

Я переписал так и все заработало, как я и хотел. Можете, пожалуйста, покритиковать с точки зрения безопасности? Буду очень благодарен.

spoiler

if ($config['recaptcha'] === true) {

            $recaptcha = $_POST['g-recaptcha-response'];
            if (!empty($recaptcha)) {

                include("curl-controller.php");
                $google_url = "https://www.google.com/recaptcha/api/siteverify";
                $secret = '********';
                $ip = $_SERVER['REMOTE_ADDR'];
                $url = $google_url . "?secret=" . $secret . "&response=" . $recaptcha . "&remoteip=" . $ip;
                $res = getCurlData($url);
                $res = json_decode($res, true);

            } else {
                $errBox = ($lang['form_err_8']);
            }

        } else {
            $res['success'] = true;
        }

      if ($res['success']) {
                   // Код формы
}

[Vitsliputsli]

zaart, $_POST['g-recaptcha-response'] вы напрямую вставляете в GET запрос, вероятно стоит валидировать эти данные перед вставкой. Или лучше использовать post запрос, для таких действий логически он больше подходит, да и secret светить в строке запроса тоже не очень хорошо.

[zaart]

Vitsliputsli,

Так будет лучше?

Код

if(isset($_POST['g-recaptcha-response']) && !empty($_POST['g-recaptcha-response'])) {

                $url = "https://www.google.com/recaptcha/api/siteverify";
                $secret = '********';
                $ip = $_SERVER['REMOTE_ADDR'];
                $data = array('secret' => $secret, 'response' => $_POST['g-recaptcha-response'], 'remoteip' => $ip);
                include("curl-controller.php");

                $res = json_decode($verifyResponse, true);

            } else {
                $errBox = ($lang['form_err_8']);
            }

        } else {
            $res['success'] = true;
        }

Сам curl.php

    $curl = curl_init($url);
    curl_setopt($curl, CURLOPT_URL, $url);
    curl_setopt($curl, CURLOPT_RETURNTRANSFER, true);
    curl_setopt($curl, CURLOPT_TIMEOUT, 10);
    curl_setopt($curl, CURLOPT_SSL_VERIFYPEER, true);
    curl_setopt($curl, CURLOPT_POST, true);
    curl_setopt($curl, CURLOPT_POSTFIELDS, http_build_query($data));
    curl_setopt($curl, CURLOPT_USERAGENT, "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/86.0.4240.75 Safari/537.36");
    $verifyResponse = curl_exec($curl);
    curl_close($curl);

[Vitsliputsli]

zaart, да, лучше. Ну еще, если есть деление на продакшен и тестовое окружение, то secret вынести в конфиг продакшена.

[zaart]

Vitsliputsli, Да, я так и сделал. Спасибо большое за помощь!

Answer 1

[v3shin]

Попробуйте такое:
$responseData = (object)['success' => true];