diff --git a/src/App.php b/src/App.php index 8d253ec76f..a48e21571d 100644 --- a/src/App.php +++ b/src/App.php @@ -695,6 +695,9 @@ class App $module = $router->getModule(); } + // Display can change depending on the requested language, so it shouldn't be cached whole + header('Vary: Accept-Language', false); + // Processes data from GET requests $httpinput = $httpInput->process(); $input = array_merge($httpinput['variables'], $httpinput['files'], $request ?? $_REQUEST); diff --git a/src/Module/ActivityPub/Objects.php b/src/Module/ActivityPub/Objects.php index f93c8b9574..34d4609511 100644 --- a/src/Module/ActivityPub/Objects.php +++ b/src/Module/ActivityPub/Objects.php @@ -46,8 +46,6 @@ class Objects extends BaseModule throw new HTTPException\BadRequestException(); } - header('Vary: Accept', false); - if (!ActivityPub::isRequest()) { DI::baseUrl()->redirect(str_replace('objects/', 'display/', DI::args()->getQueryString())); } diff --git a/src/Module/Friendica.php b/src/Module/Friendica.php index 73d5f633cd..c6e2768bd3 100644 --- a/src/Module/Friendica.php +++ b/src/Module/Friendica.php @@ -24,26 +24,44 @@ namespace Friendica\Module; use Friendica\App; use Friendica\BaseModule; use Friendica\Core\Addon; +use Friendica\Core\Config\Capability\IManageConfigValues; use Friendica\Core\Hook; +use Friendica\Core\KeyValueStorage\Capabilities\IManageKeyValuePairs; +use Friendica\Core\L10n; use Friendica\Core\Renderer; +use Friendica\Core\Session\Capability\IHandleUserSessions; use Friendica\Core\System; use Friendica\Database\PostUpdate; -use Friendica\DI; use Friendica\Model\User; use Friendica\Network\HTTPException; use Friendica\Protocol\ActivityPub; +use Friendica\Util\Profiler; +use Psr\Log\LoggerInterface; /** * Prints information about the current node - * Either in human readable form or in JSON + * Either in human-readable form or in JSON */ class Friendica extends BaseModule { + /** @var IManageConfigValues */ + private $config; + /** @var IManageKeyValuePairs */ + private $keyValue; + /** @var IHandleUserSessions */ + private $session; + + public function __construct(IHandleUserSessions $session, IManageKeyValuePairs $keyValue, IManageConfigValues $config, L10n $l10n, App\BaseURL $baseUrl, App\Arguments $args, LoggerInterface $logger, Profiler $profiler, Response $response, array $server, array $parameters = []) + { + parent::__construct($l10n, $baseUrl, $args, $logger, $profiler, $response, $server, $parameters); + + $this->config = $config; + $this->keyValue = $keyValue; + $this->session = $session; + } + protected function content(array $request = []): string { - $config = DI::config(); - $keyValue = DI::keyValue(); - $visibleAddonList = Addon::getVisibleList(); if (!empty($visibleAddonList)) { @@ -61,29 +79,30 @@ class Friendica extends BaseModule } } $addon = [ - 'title' => DI::l10n()->t('Installed addons/apps:'), + 'title' => $this->t('Installed addons/apps:'), 'list' => $sortedAddonList, ]; } else { $addon = [ - 'title' => DI::l10n()->t('No installed addons/apps'), + 'title' => $this->t('No installed addons/apps'), ]; } - $tos = ($config->get('system', 'tosdisplay')) ? - DI::l10n()->t('Read about the Terms of Service of this node.', DI::baseUrl()) : + $tos = ($this->config->get('system', 'tosdisplay')) ? + $this->t('Read about the Terms of Service of this node.', $this->baseUrl) : ''; - $blockList = $config->get('system', 'blocklist'); + $blockList = $this->config->get('system', 'blocklist') ?? []; - if (!empty($blockList)) { + $register_policy_int = $this->config->get('config', 'register_policy'); + if (!empty($blockList) && ($register_policy_int !== Register::CLOSED || $this->session->isAuthenticated())) { $blocked = [ - 'title' => DI::l10n()->t('On this server the following remote servers are blocked.'), + 'title' => $this->t('On this server the following remote servers are blocked.'), 'header' => [ - DI::l10n()->t('Blocked domain'), - DI::l10n()->t('Reason for the block'), + $this->t('Blocked domain'), + $this->t('Reason for the block'), ], - 'download' => DI::l10n()->t('Download this list in CSV format'), + 'download' => $this->t('Download this list in CSV format'), 'list' => $blockList, ]; } else { @@ -97,14 +116,14 @@ class Friendica extends BaseModule $tpl = Renderer::getMarkupTemplate('friendica.tpl'); return Renderer::replaceMacros($tpl, [ - 'about' => DI::l10n()->t('This is Friendica, version %s that is running at the web location %s. The database version is %s, the post update version is %s.', + 'about' => $this->t('This is Friendica, version %s that is running at the web location %s. The database version is %s, the post update version is %s.', '' . App::VERSION . '', - DI::baseUrl(), - '' . $config->get('system', 'build') . '/' . DB_UPDATE_VERSION . '', - '' . $keyValue->get('post_update_version') . '/' . PostUpdate::VERSION . ''), - 'friendica' => DI::l10n()->t('Please visit Friendi.ca to learn more about the Friendica project.'), - 'bugs' => DI::l10n()->t('Bug reports and issues: please visit') . ' ' . '' . DI::l10n()->t('the bugtracker at github') . '', - 'info' => DI::l10n()->t('Suggestions, praise, etc. - please email "info" at "friendi - dot - ca'), + $this->baseUrl, + '' . $this->config->get('system', 'build') . '/' . DB_UPDATE_VERSION . '', + '' . $this->keyValue->get('post_update_version') . '/' . PostUpdate::VERSION . ''), + 'friendica' => $this->t('Please visit Friendi.ca to learn more about the Friendica project.'), + 'bugs' => $this->t('Bug reports and issues: please visit') . ' ' . '' . $this->t('the bugtracker at github') . '', + 'info' => $this->t('Suggestions, praise, etc. - please email "info" at "friendi - dot - ca'), 'visible_addons' => $addon, 'tos' => $tos, @@ -115,10 +134,7 @@ class Friendica extends BaseModule protected function rawContent(array $request = []) { - header('Vary: Accept', false); - - // @TODO: Replace with parameter from router - if (DI::args()->getArgc() <= 1 || (DI::args()->getArgv()[1] !== 'json')) { + if (empty($this->parameters['format']) || $this->parameters['format'] !== 'json') { if (!ActivityPub::isRequest()) { return; } @@ -133,16 +149,14 @@ class Friendica extends BaseModule } } - $config = DI::config(); - $register_policies = [ Register::CLOSED => 'REGISTER_CLOSED', Register::APPROVE => 'REGISTER_APPROVE', Register::OPEN => 'REGISTER_OPEN' ]; - $register_policy_int = intval($config->get('config', 'register_policy')); - if ($register_policy_int !== Register::CLOSED && $config->get('config', 'invitation_only')) { + $register_policy_int = $this->config->get('config', 'register_policy'); + if ($register_policy_int !== Register::CLOSED && $this->config->get('config', 'invitation_only')) { $register_policy = 'REGISTER_INVITATION'; } else { $register_policy = $register_policies[$register_policy_int]; @@ -153,15 +167,15 @@ class Friendica extends BaseModule if (!empty($administrator)) { $admin = [ 'name' => $administrator['username'], - 'profile' => DI::baseUrl() . '/profile/' . $administrator['nickname'], + 'profile' => $this->baseUrl . '/profile/' . $administrator['nickname'], ]; } $visible_addons = Addon::getVisibleList(); - $config->reload(); + $this->config->reload(); $locked_features = []; - $featureLocks = $config->get('config', 'feature_lock'); + $featureLocks = $this->config->get('config', 'feature_lock'); if (isset($featureLocks)) { foreach ($featureLocks as $feature => $lock) { if ($feature === 'config_loaded') { @@ -174,17 +188,17 @@ class Friendica extends BaseModule $data = [ 'version' => App::VERSION, - 'url' => (string)DI::baseUrl(), + 'url' => (string)$this->baseUrl, 'addons' => $visible_addons, 'locked_features' => $locked_features, - 'explicit_content' => intval($config->get('system', 'explicit_content', 0)), - 'language' => $config->get('system', 'language'), + 'explicit_content' => intval($this->config->get('system', 'explicit_content', 0)), + 'language' => $this->config->get('system', 'language'), 'register_policy' => $register_policy, 'admin' => $admin, - 'site_name' => $config->get('config', 'sitename'), + 'site_name' => $this->config->get('config', 'sitename'), 'platform' => strtolower(App::PLATFORM), - 'info' => $config->get('config', 'info'), - 'no_scrape_url' => DI::baseUrl() . '/noscrape', + 'info' => $this->config->get('config', 'info'), + 'no_scrape_url' => $this->baseUrl . '/noscrape', ]; System::jsonExit($data); diff --git a/src/Module/Item/Display.php b/src/Module/Item/Display.php index f8e2d337bb..21046628fe 100644 --- a/src/Module/Item/Display.php +++ b/src/Module/Item/Display.php @@ -88,8 +88,6 @@ class Display extends BaseModule protected function content(array $request = []): string { - header('Vary: Accept', false); - if (ActivityPub::isRequest()) { $this->baseUrl->redirect(str_replace('display/', 'objects/', $this->args->getQueryString())); } diff --git a/src/Module/Profile/Profile.php b/src/Module/Profile/Profile.php index 31ef8d05bf..5e5028cb86 100644 --- a/src/Module/Profile/Profile.php +++ b/src/Module/Profile/Profile.php @@ -80,8 +80,6 @@ class Profile extends BaseProfile protected function rawContent(array $request = []) { - header('Vary: Accept', false); - if (ActivityPub::isRequest()) { $user = $this->database->selectFirst('user', ['uid'], ['nickname' => $this->parameters['nickname'] ?? '', 'account_removed' => false]); if ($user) { diff --git a/src/Module/Xrd.php b/src/Module/Xrd.php index 3267e9e9a2..af40bc3b02 100644 --- a/src/Module/Xrd.php +++ b/src/Module/Xrd.php @@ -38,6 +38,8 @@ class Xrd extends BaseModule { protected function rawContent(array $request = []) { + header('Vary: Accept', false); + // @TODO: Replace with parameter from router if (DI::args()->getArgv()[0] == 'xrd') { if (empty($_GET['uri'])) { diff --git a/src/Protocol/ActivityPub.php b/src/Protocol/ActivityPub.php index eaf34669ba..ad1e9367ca 100644 --- a/src/Protocol/ActivityPub.php +++ b/src/Protocol/ActivityPub.php @@ -87,6 +87,8 @@ class ActivityPub */ public static function isRequest(): bool { + header('Vary: Accept', false); + $isrequest = stristr($_SERVER['HTTP_ACCEPT'] ?? '', 'application/activity+json') || stristr($_SERVER['HTTP_ACCEPT'] ?? '', 'application/json') || stristr($_SERVER['HTTP_ACCEPT'] ?? '', 'application/ld+json'); diff --git a/static/routes.config.php b/static/routes.config.php index 7409c408d4..157f141b16 100644 --- a/static/routes.config.php +++ b/static/routes.config.php @@ -449,7 +449,7 @@ return [ '/follow_confirm' => [Module\FollowConfirm::class, [R::GET, R::POST]], '/followers/{nickname}' => [Module\ActivityPub\Followers::class, [R::GET]], '/following/{nickname}' => [Module\ActivityPub\Following::class, [R::GET]], - '/friendica[/json]' => [Module\Friendica::class, [R::GET]], + '/friendica[/{format:json}]' => [Module\Friendica::class, [R::GET]], '/friendica/inbox' => [Module\ActivityPub\Inbox::class, [R::GET, R::POST]], '/fsuggest/{contact:\d+}' => [Module\FriendSuggest::class, [R::GET, R::POST]], diff --git a/view/theme/frio/templates/nav.tpl b/view/theme/frio/templates/nav.tpl index 88a8e4bf6f..77077fdd93 100644 --- a/view/theme/frio/templates/nav.tpl +++ b/view/theme/frio/templates/nav.tpl @@ -274,8 +274,14 @@ {{/if}} +
+