Itau\API\ItauCertificate::executeRequest()

Complexity

Conditions	7
Paths	7

Size

Total Lines	55
Code Lines	33

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    private function executeRequest(string $token, string $certificadoCSR)
    {
        $headers = [
            'Content-Type: text/plain',
            'Authorization: Bearer ' . $token
        ];

        $curl = curl_init();

        curl_setopt_array($curl, [
            CURLOPT_URL => $this->endpoint,
            CURLOPT_HTTPHEADER => $headers,
            CURLOPT_RETURNTRANSFER => true,
            CURLOPT_CUSTOMREQUEST => 'POST',
            CURLOPT_POSTFIELDS => $certificadoCSR,
        ]);

        try {
            $response = curl_exec($curl);
        } catch (Exception $e) {
            curl_close($curl);
            throw new ItauException($e->getMessage(), 100);
        }

        if ($response === false) {
            $error = curl_error($curl);
            curl_close($curl);
            throw new ItauException('CURL Error: ' . $error, 100);
        }

        $statusCode = (int) curl_getinfo($curl, CURLINFO_HTTP_CODE);
        curl_close($curl);

        // Verifica status HTTP
        if ($statusCode >= 400) {
            $obj = json_decode($response);

It seems like $response can also be of type true; however, parameter $json of json_decode() does only seem to accept string, maybe add an additional type check?

            $acao = $obj->acao ?? '';
            $mensagem = $obj->mensagem ?? 'Erro desconhecido';
            $acaoText = $acao ? "- {$acao}" : '';
            throw new ItauException("HTTP Error: $statusCode - $mensagem {$acaoText}", $statusCode);
        }

        // Lógica para 204
        if ($statusCode === 204) {
            return [
                'status_code' => 204
            ];
        }

        // Verifica resposta vazia
        if (empty($response)) {
            throw new ItauException('Empty response received from server.', $statusCode);
        }

        return $response;
    }