App\Babel\Submit\Curl::post_data()

Complexity

Conditions	13
Paths	387

Size

Total Lines	40
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    protected function post_data($all_data)
    {
        if(isset($all_data["site"]))         $site = $all_data["site"];                 else throw new Exception("site is not exist in all_data");
        if(isset($all_data["data"]))         $data = $all_data["data"];                 else throw new Exception("data is not exist in all_data");
        if(isset($all_data["oj"]))           $oj = $all_data["oj"];                     else throw new Exception("oj is not exist in all_data");
        if(isset($all_data["ret"]))          $ret = $all_data["ret"];                   else $ret = false;
        if(isset($all_data["follow"]))       $follow = $all_data["follow"];             else $follow = true;
        if(isset($all_data["returnHeader"])) $returnHeader = $all_data["returnHeader"]; else $returnHeader = true;
        if(isset($all_data["postJson"]))     $postJson = $all_data["postJson"];         else $postJson = false;
        if(isset($all_data["extraHeaders"])) $extraHeaders = $all_data["extraHeaders"]; else $extraHeaders = [];
        if(isset($all_data["handle"]))       $handle = $all_data["handle"];             else $handle = "default";

        $datapost=curl_init();
        $headers=array("Expect:");
        if ($postJson) {
            $data=$data ? json_encode($data) : '{}';
            array_push($headers, 'Content-Type: application/json', 'Content-Length: '.strlen($data));
        }
        curl_setopt($datapost, CURLOPT_CAINFO, dirname(__FILE__)."/../Cookies/cacert.pem");

It seems like $datapost can also be of type false; however, parameter $ch of curl_setopt() does only seem to accept resource, maybe add an additional type check?

        curl_setopt($datapost, CURLOPT_URL, $site);
        curl_setopt($datapost, CURLOPT_HEADER, $returnHeader);
        curl_setopt($datapost, CURLOPT_HTTPHEADER, array_merge($headers, $extraHeaders));
        curl_setopt($datapost, CURLOPT_USERAGENT, "Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.109 Safari/537.36");
        curl_setopt($datapost, CURLOPT_POST, true);

        curl_setopt($datapost, CURLOPT_RETURNTRANSFER, $ret);
        curl_setopt($datapost, CURLOPT_FOLLOWLOCATION, $follow);

        curl_setopt($datapost, CURLOPT_POSTFIELDS, $data);
        curl_setopt($datapost, CURLOPT_COOKIEFILE, dirname(__FILE__)."/../Cookies/{$oj}_{$handle}.cookie");
        curl_setopt($datapost, CURLOPT_COOKIEJAR, dirname(__FILE__)."/../Cookies/{$oj}_{$handle}.cookie");
        ob_start();
        $response=curl_exec($datapost);

It seems like $datapost can also be of type false; however, parameter $ch of curl_exec() does only seem to accept resource, maybe add an additional type check?

        if (curl_errno($datapost)) {

It seems like $datapost can also be of type false; however, parameter $ch of curl_errno() does only seem to accept resource, maybe add an additional type check?

            die(curl_error($datapost));

It seems like $datapost can also be of type false; however, parameter $ch of curl_error() does only seem to accept resource, maybe add an additional type check?

Using exit here is not recommended.

        }
        ob_end_clean();
        curl_close($datapost);

It seems like $datapost can also be of type false; however, parameter $ch of curl_close() does only seem to accept resource, maybe add an additional type check?

        unset($datapost);
        return $response;
    }