page: ignore failed user setup during errors

[minimedit.git] / page.php

diff --git a/page.php b/page.php
index cdec9a802b1d55c8bbd46dcb78283e647a65125d..ed0ec1b9af93772e0b9cbbbc15e1721846ad9ffe 100644 (file)
--- a/page.php
+++ b/page.php
@@ -10,7 +10,7 @@ function abort($body, $status = NULL) {
 
 function placeholder_include($name, $params = [])
 {
-       $path = stream_resolve_include_path("$name.php");
+       $path = stream_resolve_include_path("widget/$name.php");
        if (!file_exists($path)) {
                return '<strong class="warn"><em>'.$name.'</em> ontbreekt</strong>';
        }
@@ -28,7 +28,7 @@ function placeholder_include($name, $params = [])
                }
        }
        try {
-               include "$name.php";
+               include "widget/$name.php";
                return ob_get_clean();
        }
        catch (Exception $e) {
@@ -54,8 +54,8 @@ function getoutput($blocks = [])
 
        # keep either login or logout parts depending on user level
        global $User;
-       $hideclass = empty($User) ? 'login' : 'logout';
-       $doc = preg_replace('{\s*<(p|li|span) class="'.$hideclass.'">.*?</\1>}s', '', $doc);
+       $hideclass = $User && property_exists($User, 'login') && $User->login ? 'logout' : 'login';
+       $doc = preg_replace('{\s*<([a-z]+) class="'.$hideclass.'">.*?</\1>}s', '', $doc);
 
        return preg_replace_callback(
                '{ (?<! <!--BLOCK: ) \[\[ ([^] ]+) ([^]]*) \]\] }x',
@@ -83,11 +83,17 @@ set_include_path(implode(PATH_SEPARATOR, [ DOCROOT, __DIR__ ]));
 
 function fail($error)
 {
+       global $User, $Page, $Args;
        http_response_code(500);
+       if (!isset($Article)) {
+               require_once('article.inc.php');
+               $Article = new ArchiveArticle(NULL);
+               $Article->title = 'Fout';
+       }
        include_once 'page.inc.php';
        ob_start();
-       require_once '500.html';
-       print getoutput(['debug' => $error]);
+       require '500.inc.html';
+       print getoutput(['debug' => htmlspecialchars($error)]);
 }
 
 set_exception_handler('fail');
@@ -113,42 +119,17 @@ error_reporting(error_reporting() & ~E_FATAL);
 
 # user login and control
 
-$User = NULL;
-include_once 'auth.inc.php';
+include_once 'auth.inc.php'; // sets global $User
 $Edit = isset($_GET['edit']);
 
-# distinguish subpage Args from topmost Page script
+# setup requested page
 
 $Args = '';
 $Page = preg_replace('/\?.*/', '', @$_SERVER['PATH_INFO'] ?: $_SERVER['REQUEST_URI']);
 $Page = urldecode(trim($Page, '/')) ?: 'index';
-while (TRUE) {
-       if (file_exists("$Page/.private")) {
-               # access restriction
-               if (empty($User)) {
-                       http_response_code(303);
-                       $target = urlencode($_SERVER['REQUEST_URI']);
-                       header("Location: /login?goto=$target");
-                       exit;
-               }
-               $PageAccess = $Page;
-       }
-
-       if (file_exists("$Page/index.php")) {
-               break;
-       }
-
-       $up = strrpos($Page, '/');
-       $Args = substr($Page, $up) . $Args;
-       $Page = substr($Page, 0, $up);
-       if ($up === FALSE) {
-               break;
-       }
-}
 
-$staticpage = NULL;
-if (file_exists("$Page$Args.html")) {
-       $staticpage = "$Page$Args.html";
+$staticpage = "$Page.html";
+if (file_exists($staticpage)) {
        if (is_link($staticpage)) {
                $target = preg_replace('/\.html$/', '', readlink($staticpage));
                header("HTTP/1.1 302 Shorthand");
@@ -156,49 +137,71 @@ if (file_exists("$Page$Args.html")) {
                exit;
        }
 }
-elseif (file_exists("$Page$Args/index.html")) {
-       $staticpage = "$Page$Args/index.html";
-}
-elseif ($User and $User->admin) {
-       $staticpage = (file_exists("$Page/template.html") ? "$Page/template.html" : 'template.html');
+elseif (file_exists("$Page/index.html")) {
+       $staticpage = "$Page/index.html";
 }
 
-# load static contents
-
 require_once('article.inc.php');
 $Article = new ArchiveArticle($staticpage);
 
-ob_start(); # page body
-ob_start(); # inner html
-print '<div class="static">'."\n\n";
+$Page = $Article->handler;
+$Args = $Article->path;
 
-$found = FALSE;
-if (isset($Article->raw)) {
-       print $Article->raw;
-       $found = 1;
+if ($PageAccess = $Article->restricted) {
+       # access restriction
+       if (!$User->login) {
+               http_response_code(303);
+               $target = urlencode($Article->link);
+               header("Location: /login?goto=$target");
+               exit;
+       }
 }
 
-print "</div>\n\n";
+# prepare page contents
 
-# execute dynamic code
+header(sprintf('Content-Security-Policy: %s', implode('; ', [
+       "default-src 'self' 'unsafe-inline' http://cdn.ckeditor.com", # some overrides remain
+       "img-src 'self' data: http://cdn.ckeditor.com", # inline svg (in css)
+       "base-uri 'self'", # only local pages
+       "frame-ancestors 'none'", # prevent malicious embedding
+])));
 
-$Place = [];
-
-if ($Page) {
-       $found |= require "./$Page/index.php";
-}
-
-$Place += [
-       'user'  => $User ? $User->login : '',
+ob_start(); # page body
+$Place = [
+       'user'  => $User->login ?: '',
        'url'   => htmlspecialchars($_SERVER['REQUEST_URI']),
 ];
 
-# global html
+if (isset($Article->raw)) {
+       if ($User->admin("edit $Page$Args")) {
+               # restore meta tags in static contents for editing
+               foreach (array_reverse($Article->meta) as $metaprop => $val) {
+                       $Article->raw = sprintf(
+                               '<meta property="%s" content="%s" />'."\n",
+                               $metaprop, $val
+                       ) . $Article->raw;
+               }
+       }
+}
+elseif ($User->admin("edit {$Article->link}")) {
+       $Article->raw(file_exists("$Page/template.inc.html") ? "$Page/template.inc.html" : 'template.inc.html');
+}
+if (isset($Article->raw)) {
+       $Article->raw = '<div class="static">'."\n\n".$Article->raw."</div>\n\n";
+}
+
+# output dynamic and/or static html
 
-if (!$found) {
-       # no resulting output
-       http_response_code(404);
-       @require '404.html';
+if (!$Page or require("./$Page/index.php")) {
+       # static contents
+       if (isset($Article->raw)) {
+               print $Article->raw;
+       }
+       else {
+               # no resulting output
+               http_response_code(404);
+               @require '404.inc.html';
+       }
 }
 
 include_once 'page.inc.php';