From ee35696cd42bb3b4a4515611b8a497fd2c6866da Mon Sep 17 00:00:00 2001 From: Danny Bessems Date: Tue, 18 Jun 2019 13:21:44 +0000 Subject: [PATCH] Controller for cross-domain iframes added --- include/lucidAuth.template.php | 7 +++-- public/lucidAuth.login.php | 11 +++---- public/lucidAuth.setXDomainCookie.php | 45 +++++++++++++++++---------- public/lucidAuth.validateRequest.php | 4 +-- public/misc/script.index.js | 21 +++++++++++-- 5 files changed, 58 insertions(+), 30 deletions(-) diff --git a/include/lucidAuth.template.php b/include/lucidAuth.template.php index e1b45b6..b823a3b 100644 --- a/include/lucidAuth.template.php +++ b/include/lucidAuth.template.php @@ -72,7 +72,7 @@ $pageLayout['full_alt'] = <<<'FULL_ALT' FULL_ALT; -$pageLayout['bare'] = <<<'BARE' +$pageLayout['frames'] = <<<'FRAMES' @@ -81,12 +81,13 @@ $pageLayout['bare'] = <<<'BARE' + - %1$s + %2$s -BARE; +FRAMES; $contentLayout['login'] = <<<'LOGIN' diff --git a/public/lucidAuth.login.php b/public/lucidAuth.login.php index cf55df8..d864fdb 100644 --- a/public/lucidAuth.login.php +++ b/public/lucidAuth.login.php @@ -14,10 +14,9 @@ "Result" => "Failure", "Reason" => "Failed storing authentication token in database and/or cookie" ]); -# echo '{"Result":"Fail","Reason":"Failed storing authentication token in database and/or cookie"}' . PHP_EOL; exit; } - + // Convert base64 encoded string back from JSON; // forcing it into an associative array (instead of javascript's default StdClass object) try { @@ -30,7 +29,6 @@ "Result" => "Failure", "Reason" => "Original request-URI lost in transition" ]); -# echo '{"Result":"Fail","Reason":"Original request URI lost in transition"}' . PHP_EOL; exit; } $originalUri = !empty($proxyHeaders) ? $proxyHeaders['XForwardedProto'] . '://' . $proxyHeaders['XForwardedHost'] . $proxyHeaders['XForwardedUri'] : 'lucidAuth.manage.php'; @@ -40,7 +38,8 @@ echo json_encode([ "Result" => "Success", "Location" => $originalUri, - "CrossDomainLogin" => $settings->Session['CrossDomainLogin'] + "CrossDomainLogin" => $settings->Session['CrossDomainLogin'], + "CookieDomains" => json_encode(array_diff($settings->Session['CookieDomains'], $_SERVER['HTTP_HOST'])) ]); } else { switch ($result['reason']) { @@ -63,8 +62,8 @@ } else { include_once('../include/lucidAuth.template.php'); - echo sprintf($pageLayout['full'], - sprintf($contentLayout['login'], + echo sprintf($pageLayout['full'], + sprintf($contentLayout['login'], $_GET['ref'] ) ); diff --git a/public/lucidAuth.setXDomainCookie.php b/public/lucidAuth.setXDomainCookie.php index 0d20746..9dffc41 100644 --- a/public/lucidAuth.setXDomainCookie.php +++ b/public/lucidAuth.setXDomainCookie.php @@ -4,38 +4,49 @@ include_once('../include/lucidAuth.functions.php'); // Start with checking $_REQUEST['ref'] - // What do we need? - // token again? - - // approach 1: - // origin domain, so we can intersect with $settings->Session['CookieDomains'] and iterate through the remaining domains, serving them in one page (which contains iframes already) - // this might be slower because it means one additional roundtrip between client and server - - // approach 2: - // let the client setup multiple iframes for all domains other than origin domains - // this requires passing an array of domains to the client in asynchronous reply; which feels insecure - if (!empty($_REQUEST['ref'])) { try { $queryString = json_decode(base64_decode($_REQUEST['ref']), JSON_OBJECT_AS_ARRAY); } catch (Exception $e) { // Silently fail, unless explicitly specified otherwise + header("HTTP/1.1 400 Bad Request"); if ($settings->Debug['Verbose']) throw new Exception($e); exit; } switch ($queryString['action']) { case 'login': + if (validateToken($queryString['token'])['status'] === "Success") { + // This request appears valid; store a cookie + $httpHost = $_SERVER['HTTP_HOST']; + $cookieDomain = array_values(array_filter($settings->Session['CookieDomains'], function ($value) use ($httpHost) { + // Check if $_SERVER['HTTP_HOST'] matches any of the configured domains (either explicitly or as a subdomain) + // This might seem backwards, but relying on $_SERVER directly allows spoofed values with potential security risks + return (strlen($value) > strlen($httpHost)) ? false : (0 === substr_compare($httpHost, $value, -strlen($value))); + }))[0]; + if ($cookieDomain && setcookie('JWT', $queryString['token'], (time() + $settings->Session['Duration']), '/', '.' . $cookieDomain)) { + header("HTTP/1.1 202 Accepted"); + exit; + } + else { + header("HTTP/1.1 400 Bad Request"); + exit; + } + } + else { + header("HTTP/1.1 401 Unauthorized"); + exit; + } break; default: + header("HTTP/1.1 400 Bad Request"); + exit; break; } } - - include_once('../include/lucidAuth.template.php'); - - echo sprintf($pageLayout['bare'], - '// iFrames go here' - ); + else { + header("HTTP/1.1 400 Bad Request"); + exit; + } ?> \ No newline at end of file diff --git a/public/lucidAuth.validateRequest.php b/public/lucidAuth.validateRequest.php index 289561d..ef68ff0 100644 --- a/public/lucidAuth.validateRequest.php +++ b/public/lucidAuth.validateRequest.php @@ -14,7 +14,7 @@ $proxyHeaders = array_filter($proxyHeaders, function ($key) { return substr($key, 0, 10) === 'XForwarded'; }, ARRAY_FILTER_USE_KEY); - + // For debugging purposes - enable it in ../lucidAuth.config.php if ($settings->Debug['LogToFile']) { file_put_contents('../requestHeaders.log', (new DateTime())->format('Y-m-d\TH:i:s.u') . ' --- ' . (json_encode($proxyHeaders, JSON_FORCE_OBJECT)) . PHP_EOL, FILE_APPEND); @@ -22,7 +22,7 @@ if (sizeof($proxyHeaders) === 0) { // Non-proxied request; this is senseless, go fetch! - header("HTTP/1.1 403 Forbidden"); + header("HTTP/1.1 400 Bad Request"); exit; } diff --git a/public/misc/script.index.js b/public/misc/script.index.js index 746d489..05aec5b 100644 --- a/public/misc/script.index.js +++ b/public/misc/script.index.js @@ -31,10 +31,27 @@ $(document).ready(function(){ 'color': '#FFF' }); if (data.CrossDomainLogin) { - // Create iframes for other domains console.log('CrossDomainLogin initiated'); +// For reference: +// [ +// "Result" => "Success", +// "Location" => $originalUri, +// "CrossDomainLogin" => $settings->Session['CrossDomainLogin'], +// "CookieDomains" => json_encode(array_diff($settings->Session['CookieDomains'], $_SERVER['HTTP_HOST'])) +// ]); + + var cookieDomains = json_decode(data.CookieDomains); + console.log(data.CookieDomains); +// let the client setup multiple iframes for all domains +// psuedo-code: foreach (cookieDomains as domain) { $.get(domain, ref=[action,token])} + + // validate each request against JWT-tokens in database + // load all iframes parallel, show user progress and redirect once all iframes have finished loading; or after treshold of X ms has been reached + // origin domain should be exempted from the treshold (because origin domain can be different from current domain -due to traefik design). + } -console.log("Navigating to :" + data.Location); + + // Finished (either succesfully or through timeout) cross-domain logins window.location.replace(data.Location); }, 2250); } else {