mirror of
https://github.com/silverstripe/silverstripe-framework
synced 2024-10-22 14:05:37 +02:00
FIX Flush on memory exhaustion and headers sent
This commit is contained in:
parent
5212ab031a
commit
2f9689b8f8
@ -74,9 +74,33 @@ class ErrorControlChain {
|
||||
return $error && ($error['type'] & self::$fatal_errors) != 0;
|
||||
}
|
||||
|
||||
protected function lastErrorWasMemoryExhaustion() {
|
||||
$error = error_get_last();
|
||||
$message = $error ? $error['message'] : '';
|
||||
return stripos($message, 'memory') !== false && stripos($message, 'exhausted') !== false;
|
||||
}
|
||||
|
||||
static $transtable = array(
|
||||
'k' => 1024,
|
||||
'm' => 1048576,
|
||||
'g' => 1073741824
|
||||
);
|
||||
|
||||
protected function translateMemstring($memString) {
|
||||
$char = strtolower(substr($memString, -1));
|
||||
$fact = isset(self::$transtable[$char]) ? self::$transtable[$char] : 1;
|
||||
return ((int)$memString) * $fact;
|
||||
}
|
||||
|
||||
public function handleFatalError() {
|
||||
if ($this->handleFatalErrors && $this->suppression) {
|
||||
if ($this->lastErrorWasFatal()) {
|
||||
if ($this->lastErrorWasMemoryExhaustion()) {
|
||||
// Bump up memory limit by an arbitrary 10% / 10MB (whichever is bigger) since we've run out
|
||||
$cur = $this->translateMemstring(ini_get('memory_limit'));
|
||||
if ($cur != -1) ini_set('memory_limit', $cur + max(round($cur*0.1), 10000000));
|
||||
}
|
||||
|
||||
$this->error = true;
|
||||
$this->step();
|
||||
}
|
||||
|
@ -94,7 +94,14 @@ class ParameterConfirmationToken {
|
||||
$location = "$proto://" . $host . BASE_URL . $url . ($params ? '?'.http_build_query($params) : '');
|
||||
|
||||
// And redirect
|
||||
header('location: '.$location, true, 302);
|
||||
if (headers_sent()) {
|
||||
echo "
|
||||
<script>location.href='$location';</script>
|
||||
<noscript><meta http-equiv='refresh' content='0; url=$location'></noscript>
|
||||
You are being redirected. If you are not redirected soon, <a href='$location'>click here to continue the flush</a>
|
||||
";
|
||||
}
|
||||
else header('location: '.$location, true, 302);
|
||||
die;
|
||||
}
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user