OpenEcoLab-LasAlpujarras
/
farmOS
mirror of https://github.com/farmOS/farmOS.git
3
0
Fork 0
Code Wiki

Limit the maximum number of data points to 100k via API.

This commit is contained in:
Michael Stenta 2020-08-26 16:39:28 -04:00
parent d6e7a3deb4
commit f0bc6a8e6f
1 changed files with 13 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
modules/farm/farm_sensor/farm_sensor_listener/farm_sensor_listener.module
View File

@ -187,6 +187,19 @@ function farm_sensor_listener_page_callback($public_key) {
$limit = 1;
if (isset($params['limit'])) {
$limit = $params['limit'];
// On second thought... only allow 100k max data points. Otherwise, it's
// possible to exhaust PHP's memory, which is a potential DDoS vector.
// If the requested limit is 0, set it to the max automatically.
// If more than the max is specified, return a 422 error code.
$max = 100000;
if ($limit == 0) {
$limit = $max;
}
elseif ($limit > $max) {
drupal_add_http_header('Status', '422 Unprocessable Entity: exceeds max limit of ' . $max);
return MENU_FOUND;
}
}
// If the 'offset' parameter is set, offset the results.