afterlogic / webmail-lite-8

Open-source webmail script for existing IMAP server
https://afterlogic.org/webmail-lite-8
GNU Affero General Public License v3.0
335 stars 71 forks source link

More sensible error handling please #9

Closed udf2457 closed 6 years ago

udf2457 commented 6 years ago

At the moment, errors such as the one I previously reported are blurted out for the whole world to see.

Surely a more sensible approach would be the following concept:

if ($verbose_output_enabled) {
     error_reporting (E_ALL | E_STRICT);
     ini_set ('display_errors', 1);
} else {
      error_reporting (E_ALL ^ E_NOTICE | E_STRICT);
      ini_set ('display_errors', 0);
}
afterlogic-support commented 6 years ago

We'll have it considered, thank you.

afterlogic-support commented 5 years ago

Developers have added a special GET/POST parameter dbg which enables displaying errors. For example: https://webmail.yourdomain.com/?dbg For POST requests, the parameter can be added to RequestBody.