diff --git a/mod/fbrowser.php b/mod/fbrowser.php index 1646b19dff..81284d6b91 100644 --- a/mod/fbrowser.php +++ b/mod/fbrowser.php @@ -39,11 +39,11 @@ use Friendica\Util\Strings; function fbrowser_content(App $a) { if (!local_user()) { - exit(); + System::exit(); } if (DI::args()->getArgc() == 1) { - exit(); + System::exit(); } // Needed to match the correct template in a module that uses a different theme than the user/site/default diff --git a/mod/photos.php b/mod/photos.php index ce6d11340e..52d749bb46 100644 --- a/mod/photos.php +++ b/mod/photos.php @@ -158,7 +158,7 @@ function photos_post(App $a) if (!$can_post) { notice(DI::l10n()->t('Permission denied.')); - exit(); + System::exit(); } $owner_record = User::getOwnerDataById($page_owner_uid); @@ -166,7 +166,7 @@ function photos_post(App $a) if (!$owner_record) { notice(DI::l10n()->t('Contact information unavailable')); DI::logger()->info('photos_post: unable to locate contact record for page owner. uid=' . $page_owner_uid); - exit(); + System::exit(); } $aclFormatter = DI::aclFormatter(); diff --git a/mod/pubsub.php b/mod/pubsub.php index e07b15583c..918c55ee14 100644 --- a/mod/pubsub.php +++ b/mod/pubsub.php @@ -22,6 +22,7 @@ use Friendica\App; use Friendica\Core\Logger; use Friendica\Core\Protocol; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Model\Contact; @@ -38,7 +39,7 @@ function hub_return($valid, $body) } else { throw new \Friendica\Network\HTTPException\NotFoundException(); } - exit(); + System::exit(); } // when receiving an XML feed, always return OK diff --git a/mod/pubsubhubbub.php b/mod/pubsubhubbub.php index 361cb0a597..5dda9ee0af 100644 --- a/mod/pubsubhubbub.php +++ b/mod/pubsubhubbub.php @@ -21,6 +21,7 @@ use Friendica\App; use Friendica\Core\Logger; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Model\PushSubscriber; @@ -142,5 +143,5 @@ function pubsubhubbub_init(App $a) { throw new \Friendica\Network\HTTPException\AcceptedException(); } - exit(); + System::exit(); } diff --git a/mod/share.php b/mod/share.php index b18655aef8..2f836c5aa0 100644 --- a/mod/share.php +++ b/mod/share.php @@ -21,6 +21,7 @@ use Friendica\App; use Friendica\Content\Text\BBCode; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Model\Item; @@ -30,7 +31,7 @@ function share_init(App $a) { $post_id = ((DI::args()->getArgc() > 1) ? intval(DI::args()->getArgv()[1]) : 0); if (!$post_id || !local_user()) { - exit(); + System::exit(); } $fields = ['private', 'body', 'author-name', 'author-link', 'author-avatar', @@ -38,7 +39,7 @@ function share_init(App $a) { $item = Post::selectFirst($fields, ['id' => $post_id]); if (!DBA::isResult($item) || $item['private'] == Item::PRIVATE) { - exit(); + System::exit(); } if (strpos($item['body'], "[/share]") !== false) { @@ -56,5 +57,5 @@ function share_init(App $a) { } echo $o; - exit(); + System::exit(); } diff --git a/mod/tagger.php b/mod/tagger.php index 310e3c6140..1acce4b0b6 100644 --- a/mod/tagger.php +++ b/mod/tagger.php @@ -167,6 +167,5 @@ EOT; $post = Post::selectFirst(['uri-id', 'uid'], ['id' => $post_id]); Worker::add(PRIORITY_HIGH, "Notifier", Delivery::POST, $post['uri-id'], $post['uid']); - - exit(); + System::exit(); } diff --git a/mod/wall_attach.php b/mod/wall_attach.php index 607d0fe7a3..43f4681df3 100644 --- a/mod/wall_attach.php +++ b/mod/wall_attach.php @@ -67,14 +67,14 @@ function wall_attach_post(App $a) { System::jsonExit(['error' => DI::l10n()->t('Permission denied.')]); } notice(DI::l10n()->t('Permission denied.') . EOL ); - exit(); + System::exit(); } if (empty($_FILES['userfile'])) { if ($r_json) { System::jsonExit(['error' => DI::l10n()->t('Invalid request.')]); } - exit(); + System::exit(); } $src = $_FILES['userfile']['tmp_name']; @@ -97,7 +97,7 @@ function wall_attach_post(App $a) { } else { notice($msg); } - exit(); + System::exit(); } if ($maxfilesize && $filesize > $maxfilesize) { @@ -108,7 +108,7 @@ function wall_attach_post(App $a) { } else { echo $msg . EOL; } - exit(); + System::exit(); } $newid = Attach::storeFile($src, $page_owner_uid, $filename, '<' . $page_owner_cid . '>'); @@ -122,7 +122,7 @@ function wall_attach_post(App $a) { } else { echo $msg . EOL; } - exit(); + System::exit(); } if ($r_json) { @@ -132,7 +132,6 @@ function wall_attach_post(App $a) { $lf = "\n"; echo $lf . $lf . '[attachment]' . $newid . '[/attachment]' . $lf; - - exit(); + System::exit(); // NOTREACHED } diff --git a/mod/wall_upload.php b/mod/wall_upload.php index 79f1dcc41f..36e313cece 100644 --- a/mod/wall_upload.php +++ b/mod/wall_upload.php @@ -89,14 +89,14 @@ function wall_upload_post(App $a, $desktopmode = true) System::jsonExit(['error' => DI::l10n()->t('Permission denied.')]); } notice(DI::l10n()->t('Permission denied.')); - exit(); + System::exit(); } if (empty($_FILES['userfile']) && empty($_FILES['media'])) { if ($r_json) { System::jsonExit(['error' => DI::l10n()->t('Invalid request.')]); } - exit(); + System::exit(); } $src = ''; @@ -148,7 +148,7 @@ function wall_upload_post(App $a, $desktopmode = true) System::jsonExit(['error' => DI::l10n()->t('Invalid request.')]); } notice(DI::l10n()->t('Invalid request.')); - exit(); + System::exit(); } $filetype = Images::getMimeTypeBySource($src, $filename, $filetype); @@ -167,7 +167,7 @@ function wall_upload_post(App $a, $desktopmode = true) } else { echo $msg. EOL; } - exit(); + System::exit(); } $Image->orient($src); @@ -205,7 +205,7 @@ function wall_upload_post(App $a, $desktopmode = true) } else { echo $msg. EOL; } - exit(); + System::exit(); } } @@ -229,7 +229,7 @@ function wall_upload_post(App $a, $desktopmode = true) } else { echo $msg. EOL; } - exit(); + System::exit(); } if ($width > 640 || $height > 640) { @@ -281,6 +281,6 @@ function wall_upload_post(App $a, $desktopmode = true) } echo "\n\n" . '[url=' . DI::baseUrl() . '/photos/' . $page_owner_nick . '/image/' . $resource_id . '][img]' . DI::baseUrl() . "/photo/{$resource_id}-{$smallest}.".$Image->getExt()."[/img][/url]\n\n"; - exit(); + System::exit(); // NOTREACHED } diff --git a/src/App.php b/src/App.php index f7c929820d..07296cecf0 100644 --- a/src/App.php +++ b/src/App.php @@ -576,6 +576,7 @@ class App $this->profiler->set(microtime(true), 'classinit'); $moduleName = $this->args->getModuleName(); + $page->setLogging($this->args->getCommand(), $this->args->getMethod()); try { // Missing DB connection: ERROR @@ -718,6 +719,7 @@ class App } catch (HTTPException $e) { (new ModuleHTTPException())->rawContent($e); } + $page->logRuntime(); } /** diff --git a/src/App/Page.php b/src/App/Page.php index 14f0101ef0..305b2962b9 100644 --- a/src/App/Page.php +++ b/src/App/Page.php @@ -30,6 +30,7 @@ use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\PConfig\Capability\IManagePersonalConfigValues; use Friendica\Core\Hook; use Friendica\Core\L10n; +use Friendica\Core\Logger; use Friendica\Core\Renderer; use Friendica\Core\System; use Friendica\Core\Theme; @@ -78,14 +79,31 @@ class Page implements ArrayAccess */ private $basePath; + private $timestamp = 0; + private $command = ''; + private $method = ''; + /** * @param string $basepath The Page basepath */ public function __construct(string $basepath) { + $this->timestamp = microtime(true); $this->basePath = $basepath; } + public function setLogging(string $command, string $method) + { + $this->command = $command; + $this->method = $method; + } + + public function logRuntime() + { + $runtime = number_format(microtime(true) - $this->timestamp, 3); + Logger::debug('Runtime', ['method' => $this->method, 'command' => $this->command, 'runtime' => $runtime]); + } + /** * Whether a offset exists * @@ -421,6 +439,9 @@ class Page implements ArrayAccess { $moduleName = $args->getModuleName(); + $this->command = $moduleName; + $this->method = $args->getMethod(); + /* Create the page content. * Calls all hooks which are including content operations * diff --git a/src/Core/System.php b/src/Core/System.php index 66b6b25a50..10fc5c7d4e 100644 --- a/src/Core/System.php +++ b/src/Core/System.php @@ -295,7 +295,7 @@ class System DI::apiResponse()->addContent(XML::fromArray(["result" => $result], $xml)); DI::page()->exit(DI::apiResponse()->generate()); - exit(); + self::exit(); } /** @@ -315,7 +315,7 @@ class System DI::apiResponse()->addContent($content); DI::page()->exit(DI::apiResponse()->generate()); - exit(); + self::exit(); } /** @@ -331,7 +331,8 @@ class System DI::apiResponse()->setType($responce, $content_type); DI::apiResponse()->addContent($content); DI::page()->exit(DI::apiResponse()->generate()); - exit(); + + self::exit(); } public static function jsonError($httpCode, $content, $content_type = 'application/json') @@ -359,6 +360,16 @@ class System DI::apiResponse()->setType(Response::TYPE_JSON, $content_type); DI::apiResponse()->addContent(json_encode($content, $options)); DI::page()->exit(DI::apiResponse()->generate()); + + self::exit(); + } + + /** + * Exit the program execution. + */ + public static function exit() + { + DI::page()->logRuntime(); exit(); } @@ -448,8 +459,7 @@ class System case 307: throw new TemporaryRedirectException(); } - - exit(); + self::exit(); } /** @@ -522,7 +532,7 @@ class System echo str_replace("\t", " ", $o); echo ""; echo "\r\n"; - exit(); + self::exit(); } /** diff --git a/src/Module/Admin/PhpInfo.php b/src/Module/Admin/PhpInfo.php index 0177424f3d..c51ed27c01 100644 --- a/src/Module/Admin/PhpInfo.php +++ b/src/Module/Admin/PhpInfo.php @@ -21,6 +21,7 @@ namespace Friendica\Module\Admin; +use Friendica\Core\System; use Friendica\Module\BaseAdmin; class PhpInfo extends BaseAdmin @@ -30,6 +31,6 @@ class PhpInfo extends BaseAdmin self::checkAdminAccess(); phpinfo(); - exit(); + System::exit(); } } diff --git a/src/Module/Attach.php b/src/Module/Attach.php index 60d1015b0a..a73beb2b8d 100644 --- a/src/Module/Attach.php +++ b/src/Module/Attach.php @@ -23,6 +23,7 @@ namespace Friendica\Module; use Friendica\BaseModule; use Friendica\Core\Logger; +use Friendica\Core\System; use Friendica\DI; use Friendica\Model\Attach as MAttach; @@ -72,7 +73,7 @@ class Attach extends BaseModule } echo $data; - exit(); + System::exit(); // NOTREACHED } } diff --git a/src/Module/Contact/Hovercard.php b/src/Module/Contact/Hovercard.php index cd03e2533a..440c46c502 100644 --- a/src/Module/Contact/Hovercard.php +++ b/src/Module/Contact/Hovercard.php @@ -24,6 +24,7 @@ namespace Friendica\Module\Contact; use Friendica\BaseModule; use Friendica\Core\Renderer; use Friendica\Core\Session; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Model\Contact; @@ -107,6 +108,6 @@ class Hovercard extends BaseModule ]); echo $o; - exit(); + System::exit(); } } diff --git a/src/Module/Debug/ItemBody.php b/src/Module/Debug/ItemBody.php index 55abc894ed..a9a3c0b50b 100644 --- a/src/Module/Debug/ItemBody.php +++ b/src/Module/Debug/ItemBody.php @@ -22,6 +22,7 @@ namespace Friendica\Module\Debug; use Friendica\BaseModule; +use Friendica\Core\System; use Friendica\DI; use Friendica\Model\Post; use Friendica\Network\HTTPException; @@ -48,7 +49,7 @@ class ItemBody extends BaseModule if (!empty($item)) { if (DI::mode()->isAjax()) { echo str_replace("\n", '
', $item['body']); - exit(); + System::exit(); } else { return str_replace("\n", '
', $item['body']); } diff --git a/src/Module/Group.php b/src/Module/Group.php index 3cd8166f1f..cd07e97812 100644 --- a/src/Module/Group.php +++ b/src/Module/Group.php @@ -360,7 +360,7 @@ class Group extends BaseModule if ($change) { $tpl = Renderer::getMarkupTemplate('groupeditor.tpl'); echo Renderer::replaceMacros($tpl, $context); - exit(); + System::exit(); } return Renderer::replaceMacros($tpl, $context); diff --git a/src/Module/HTTPException/PageNotFound.php b/src/Module/HTTPException/PageNotFound.php index a73757d828..4cfa35276d 100644 --- a/src/Module/HTTPException/PageNotFound.php +++ b/src/Module/HTTPException/PageNotFound.php @@ -22,6 +22,7 @@ namespace Friendica\Module\HTTPException; use Friendica\BaseModule; +use Friendica\Core\System; use Friendica\DI; use Friendica\Network\HTTPException; use Psr\Http\Message\ResponseInterface; @@ -47,7 +48,7 @@ class PageNotFound extends BaseModule $queryString = $this->server['QUERY_STRING']; // Stupid browser tried to pre-fetch our Javascript img template. Don't log the event or return anything - just quietly exit. if (!empty($queryString) && preg_match('/{[0-9]}/', $queryString) !== 0) { - exit(); + System::exit(); } if (!empty($queryString) && ($queryString === 'q=internal_error.html') && isset($dreamhost_error_hack)) { diff --git a/src/Module/Oembed.php b/src/Module/Oembed.php index 81c858cba3..be8168271b 100644 --- a/src/Module/Oembed.php +++ b/src/Module/Oembed.php @@ -23,6 +23,7 @@ namespace Friendica\Module; use Friendica\BaseModule; use Friendica\Content; +use Friendica\Core\System; use Friendica\DI; use Friendica\Util\Strings; @@ -43,14 +44,14 @@ class Oembed extends BaseModule if (DI::args()->getArgv()[1] == 'b2h') { $url = ["", trim(hex2bin($_REQUEST['url']))]; echo Content\OEmbed::replaceCallback($url); - exit(); + System::exit(); } // Unused form: /oembed/h2b?text=... if (DI::args()->getArgv()[1] == 'h2b') { $text = trim(hex2bin($_REQUEST['text'])); echo Content\OEmbed::HTML2BBCode($text); - exit(); + System::exit(); } // @TODO: Replace with parameter from router @@ -68,6 +69,6 @@ class Oembed extends BaseModule echo $j->html; echo ''; } - exit(); + System::exit(); } } diff --git a/src/Module/PermissionTooltip.php b/src/Module/PermissionTooltip.php index ff0bae1bc4..07faa72e3e 100644 --- a/src/Module/PermissionTooltip.php +++ b/src/Module/PermissionTooltip.php @@ -22,6 +22,7 @@ namespace Friendica\Module; use Friendica\Core\Hook; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\DI; use Friendica\Model\APContact; @@ -164,8 +165,7 @@ class PermissionTooltip extends \Friendica\BaseModule } else { echo $o . $receivers; } - - exit(); + System::exit(); } /** diff --git a/src/Module/Photo.php b/src/Module/Photo.php index 9e56301b44..134cade650 100644 --- a/src/Module/Photo.php +++ b/src/Module/Photo.php @@ -32,6 +32,7 @@ use Friendica\Model\Post; use Friendica\Model\Profile; use Friendica\Core\Storage\Type\ExternalResource; use Friendica\Core\Storage\Type\SystemResource; +use Friendica\Core\System; use Friendica\Core\Worker; use Friendica\Model\User; use Friendica\Network\HTTPClient\Client\HttpClientAccept; @@ -224,7 +225,7 @@ class Photo extends BaseModule 'output' => number_format($output, 3), 'rest' => number_format($rest, 3)]); } - exit(); + System::exit(); } private static function getPhotoByid(int $id, $type, $customsize) diff --git a/src/Module/Proxy.php b/src/Module/Proxy.php index 3ef3117d69..90f1185800 100644 --- a/src/Module/Proxy.php +++ b/src/Module/Proxy.php @@ -202,6 +202,6 @@ class Proxy extends BaseModule header('Expires: ' . gmdate('D, d M Y H:i:s', time() + (31536000)) . ' GMT'); header('Cache-Control: max-age=31536000'); echo $img->asString(); - exit(); + System::exit(); } } diff --git a/src/Module/RobotsTxt.php b/src/Module/RobotsTxt.php index 1fcf1269ed..3651382a00 100644 --- a/src/Module/RobotsTxt.php +++ b/src/Module/RobotsTxt.php @@ -22,6 +22,7 @@ namespace Friendica\Module; use Friendica\BaseModule; +use Friendica\Core\System; /** * Return the default robots.txt @@ -44,6 +45,6 @@ class RobotsTxt extends BaseModule foreach ($allDisalloweds as $disallowed) { echo 'Disallow: ' . $disallowed . PHP_EOL; } - exit(); + System::exit(); } } diff --git a/src/Module/Settings/UserExport.php b/src/Module/Settings/UserExport.php index 1d47df4fd3..2d29f4d647 100644 --- a/src/Module/Settings/UserExport.php +++ b/src/Module/Settings/UserExport.php @@ -23,6 +23,7 @@ namespace Friendica\Module\Settings; use Friendica\Core\Hook; use Friendica\Core\Renderer; +use Friendica\Core\System; use Friendica\Database\DBA; use Friendica\Database\DBStructure; use Friendica\DI; @@ -112,8 +113,7 @@ class UserExport extends BaseSettings self::exportContactsAsCSV(local_user()); break; } - - exit(); + System::exit(); } } diff --git a/src/Module/Theme.php b/src/Module/Theme.php index 66c9d06f07..50a4f6c55a 100644 --- a/src/Module/Theme.php +++ b/src/Module/Theme.php @@ -22,7 +22,7 @@ namespace Friendica\Module; use Friendica\BaseModule; -use Friendica\DI; +use Friendica\Core\System; use Friendica\Util\Strings; /** @@ -45,7 +45,6 @@ class Theme extends BaseModule if (file_exists("view/theme/$theme/style.php")) { require_once "view/theme/$theme/style.php"; } - - exit(); + System::exit(); } } diff --git a/src/Module/ThemeDetails.php b/src/Module/ThemeDetails.php index 6398a00c99..1b12f3e37a 100644 --- a/src/Module/ThemeDetails.php +++ b/src/Module/ThemeDetails.php @@ -48,6 +48,6 @@ class ThemeDetails extends BaseModule 'credits' => $credits, ]); } - exit(); + System::exit(); } } diff --git a/src/Module/Update/Network.php b/src/Module/Update/Network.php index 330cba2d3e..6282028e69 100644 --- a/src/Module/Update/Network.php +++ b/src/Module/Update/Network.php @@ -31,7 +31,7 @@ class Network extends NetworkModule protected function rawContent(array $request = []) { if (!isset($_GET['p']) || !isset($_GET['item'])) { - exit(); + System::exit(); } $this->parseRequest($_GET); diff --git a/src/Util/JsonLD.php b/src/Util/JsonLD.php index cf1e058a14..732b4b985d 100644 --- a/src/Util/JsonLD.php +++ b/src/Util/JsonLD.php @@ -24,6 +24,7 @@ namespace Friendica\Util; use Friendica\Core\Cache\Enum\Duration; use Friendica\Core\Logger; use Exception; +use Friendica\Core\System; use Friendica\DI; /** @@ -69,7 +70,7 @@ class JsonLD if ($recursion > 5) { Logger::error('jsonld bomb detected at: ' . $url); - exit(); + System::exit(); } $result = DI::cache()->get('documentLoader:' . $url);