Return http status error on exit #15

New issue

Closed

opened 2022-01-31 15:44:07 +01:00 by Cpm · 1 comment

Cpm commented

2022-01-31 15:44:07 +01:00

Contributor

Actually, the index.php file is containing some tests in case of qrcode creation POST request:

  if (strlen($_POST['txt']) >= 1 AND strlen($_POST['txt']) <= 4096)
    $params['txt'] = $_POST['txt'];
  else
    exit("Wrong value for txt");

But, on error, the http status code returned is 200 OK.

It would be better if it returned the http status code 400 Bad Request in case of error.

Example of code:

  if (strlen($_POST['txt']) >= 1 AND strlen($_POST['txt']) <= 4096)
    $params['txt'] = $_POST['txt'];
  else
  {
      http_response_code(400);
      exit("Wrong value for txt");     
  }

This need is very important to do log analysis.

Actually, the `index.php` file is containing some tests in case of qrcode creation POST request: ``` if (strlen($_POST['txt']) >= 1 AND strlen($_POST['txt']) <= 4096) $params['txt'] = $_POST['txt']; else exit("Wrong value for txt"); ``` But, on error, the http status code returned is `200 OK`. It would be better if it returned the http status code `400 Bad Request` in case of error. Example of code: ``` if (strlen($_POST['txt']) >= 1 AND strlen($_POST['txt']) <= 4096) $params['txt'] = $_POST['txt']; else { http_response_code(400); exit("Wrong value for txt"); } ``` This need is very important to do log analysis.

👍 1