Skip to content

Do not respond with 200 OK if proxied request fails #41

Open
@avbentem

Description

@avbentem

Currently fatal errors such as invalid domains simply return 200 OK. To mitigate that one could, e.g., throw 502 Bad Gateway instead.

Change:

// retrieve response (headers and content)
$response = curl_exec($ch);
curl_close($ch);

...into:

// retrieve response (headers and content)
$response = curl_exec($ch);
if ($response === false) {
    http_response_code(502);
    echo 'Failed to proxy to ' . $request_url . ': ' . curl_error($ch);
    curl_close($ch);
    exit();
}
curl_close($ch);

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions