diff --git a/nodejs/app.js b/nodejs/app.js index d4c3143..faeb43a 100755 --- a/nodejs/app.js +++ b/nodejs/app.js @@ -61,6 +61,11 @@ app.set('trust proxy', 1); app.set('views', path.join(__dirname, 'views')); app.set('view engine', 'ejs'); +// Per-app values for the shared UI shell (views/top.ejs + views/bottom.ejs). +// Set as an app local so every res.render has it, including routes that don't +// spread the routers' `values` object. +app.locals.ui = require('./utils/ui'); + // Have express server static content( images, CSS, browser JS) from the public // local folder. maxAge is short since this is the app's own JS/CSS, which // changes on every deploy and isn't cache-busted/fingerprinted. diff --git a/nodejs/package-lock.json b/nodejs/package-lock.json index 385b615..3a54984 100644 --- a/nodejs/package-lock.json +++ b/nodejs/package-lock.json @@ -1,12 +1,12 @@ { "name": "t42-sso-manager", - "version": "1.3.2", + "version": "1.4.0", "lockfileVersion": 3, "requires": true, "packages": { "": { "name": "t42-sso-manager", - "version": "1.3.2", + "version": "1.4.0", "license": "MIT", "dependencies": { "@fortawesome/fontawesome-free": "^7.3.0", @@ -24,7 +24,7 @@ "express-rate-limit": "^8.5.2", "extend": "^3.0.2", "jq-repeat": "^2.2.0", - "jquery": "^3.7.1", + "jquery": "^4.0.0", "jsonwebtoken": "^9.0.3", "ldapts": "^8.1.8", "lru-cache": "^11.5.1", @@ -4853,9 +4853,9 @@ } }, "node_modules/jquery": { - "version": "3.7.1", - "resolved": "https://registry.npmjs.org/jquery/-/jquery-3.7.1.tgz", - "integrity": "sha512-m4avr8yL8kmFN8psrbFFFmB/If14iN5o9nw/NgnnM+kybDJpRsAynV2BsfpTYrTRysYUdADVD7CkUUizgkpLfg==", + "version": "4.0.0", + "resolved": "https://registry.npmjs.org/jquery/-/jquery-4.0.0.tgz", + "integrity": "sha512-TXCHVR3Lb6TZdtw1l3RTLf8RBWVGexdxL6AC8/e0xZKEpBflBsjh9/8LXw+dkNFuOyW9B7iB3O1sP7hS0Kiacg==", "license": "MIT" }, "node_modules/js-tokens": { diff --git a/nodejs/package.json b/nodejs/package.json index 948511f..7e0630f 100755 --- a/nodejs/package.json +++ b/nodejs/package.json @@ -36,7 +36,7 @@ "express-rate-limit": "^8.5.2", "extend": "^3.0.2", "jq-repeat": "^2.2.0", - "jquery": "^3.7.1", + "jquery": "^4.0.0", "jsonwebtoken": "^9.0.3", "ldapts": "^8.1.8", "lru-cache": "^11.5.1", diff --git a/nodejs/public/js/app.js b/nodejs/public/js/app.js index c77cb04..97d7e3a 100755 --- a/nodejs/public/js/app.js +++ b/nodejs/public/js/app.js @@ -396,7 +396,7 @@ app.impersonate = (function(app){ app.token = (function(app){ function list(name, callack){ - if($.isFunction(name)){ + if(typeof name === 'function'){ callack = name; name = ''; } diff --git a/nodejs/public/lib/js/app-base.js b/nodejs/public/lib/js/app-base.js index 125717b..2b323a3 100644 --- a/nodejs/public/lib/js/app-base.js +++ b/nodejs/public/lib/js/app-base.js @@ -1,3 +1,12 @@ +// Shared client framework for the theta42 apps. +// +// This file is byte-identical across sso-manager-node, proxy and jump-host — +// per-app behaviour comes from the server (the `ui` locals in views/top.ejs and +// the /api/user/me response), never from edits to this file. Edit all three +// copies together. +// +// jQuery 4 safe: no $.isFunction, no $.holdReady. + var app = {}; app.pubsub = (function(){ @@ -45,7 +54,7 @@ app.pubsub = (function(){ app.socket = (function(app){ // $.getScript('/socket.io/socket.io.js') // - + var socket; $(document).ready(function(){ socket = io({ @@ -75,11 +84,17 @@ app.socket = (function(app){ app.api = (function(app){ var baseURL = '/api/' - function post(url, data, callback){ - if (!$.isFunction(callback)) { - return new Promise((resolve, reject) => { + // post/put/delete are dual-mode: pass a callback for the node-style + // (error, data, status) form, or omit it to get a Promise that resolves + // with the parsed body and rejects with the error body. get/options return + // the jqXHR, which is itself thenable, so `await app.api.get(...)` works. + + function body(method, url, data, callback){ + if(typeof callback !== 'function'){ + return new Promise(function(resolve, reject){ $.ajax({ - type: 'POST', url: baseURL+url, + type: method, + url: baseURL+url, headers: { 'auth-token': app.auth.getToken() }, data: JSON.stringify(data), contentType: 'application/json; charset=utf-8', @@ -88,9 +103,11 @@ app.api = (function(app){ }); } return $.ajax({ - type: 'POST', + type: method, url: baseURL+url, - headers:{ 'auth-token': app.auth.getToken() }, + headers:{ + 'auth-token': app.auth.getToken() + }, data: JSON.stringify(data), contentType: "application/json; charset=utf-8", dataType: "json", @@ -104,40 +121,20 @@ app.api = (function(app){ }); } + function post(url, data, callback){ + return body('POST', url, data, callback); + } + function put(url, data, callback){ - if (!$.isFunction(callback)) { - return new Promise((resolve, reject) => { - $.ajax({ - type: 'PUT', url: baseURL+url, - headers: { 'auth-token': app.auth.getToken() }, - data: JSON.stringify(data), - contentType: 'application/json; charset=utf-8', - dataType: 'json', - }).done(resolve).fail(function(xhr){ reject(xhr.responseJSON || {}); }); - }); - } - return $.ajax({ - type: 'PUT', - url: baseURL+url, - headers:{ 'auth-token': app.auth.getToken() }, - data: JSON.stringify(data), - contentType: "application/json; charset=utf-8", - dataType: "json", - complete: function(res, text){ - callback( - text !== 'success' ? res.statusText : null, - JSON.parse(res.responseText), - res.status - ); - } - }); + return body('PUT', url, data, callback); } function remove(url, callback){ - if (!$.isFunction(callback)) { - return new Promise((resolve, reject) => { + if(typeof callback !== 'function'){ + return new Promise(function(resolve, reject){ $.ajax({ - type: 'DELETE', url: baseURL+url, + type: 'DELETE', + url: baseURL+url, headers: { 'auth-token': app.auth.getToken() }, contentType: 'application/json; charset=utf-8', dataType: 'json', @@ -147,7 +144,9 @@ app.api = (function(app){ return $.ajax({ type: 'DELETE', url: baseURL+url, - headers:{ 'auth-token': app.auth.getToken() }, + headers:{ + 'auth-token': app.auth.getToken() + }, contentType: "application/json; charset=utf-8", dataType: "json", complete: function(res, text){ @@ -202,7 +201,10 @@ app.api = (function(app){ })(app) app.auth = (function(app){ - var user = {}; + // One in-flight/cached GET /api/user/me per page load. Every gating + // decision (nav items, per-view forceLogin, group-required elements) reads + // this same promise instead of re-fetching. + var userPromise = null; function setToken(token){ localStorage.setItem('APIToken', token); @@ -216,35 +218,70 @@ app.auth = (function(app){ try{ return await app.api.get('user/me'); }catch(error){ - if(error?.status === 401) return null; - throw error + if(error && error.status === 401) return null; + throw error; } } + // Cached current user, or false when there's no token at all. Callers that + // need a fresh copy (after a login or a profile change) pass force. + function loadUser(force){ + if(force || !userPromise){ + userPromise = getToken() ? getUser() : Promise.resolve(null); + userPromise = userPromise.then(function(user){ + app.auth.user = app.auth.perms = user || null; + return user; + }); + } + return userPromise; + } + + // The apps report group membership two ways: sso-manager-node returns LDAP + // DNs in `memberOf`, the OIDC clients return plain CNs in `groups`. Both + // normalise to a list of CNs. `isAdmin` (the clients' effective-rights flag) + // is exposed as a synthetic `admin` group so one gating model covers both. + function groupCNs(user){ + var raw = (user && (user.memberOf || user.groups)) || []; + if(!Array.isArray(raw)) raw = [raw]; + var names = raw.map(function(group){ + return String(group).split(',')[0].replace(/^cn=/i, ''); + }); + if(user && user.isAdmin && names.indexOf('admin') === -1) names.push('admin'); + return names; + } + async function memberOf(groupNameToFind, user){ - try{ - user = user || await app.auth.asyncUser; - groupNameToFind = Array.isArray(groupNameToFind) ? groupNameToFind : [groupNameToFind] + user = user || await loadUser(); + if(!user) return false; + groupNameToFind = Array.isArray(groupNameToFind) ? groupNameToFind : [groupNameToFind]; - for(let group of user.memberOf){ - group = group.split(',ou=groups')[0].replace('cn=', ''); - if(groupNameToFind.includes(group)) return true; - } - - return false; - - }catch(error){ - throw(error); - } + return groupCNs(user).some(function(group){ + return groupNameToFind.includes(group); + }); } - async function isLoggedIn(){ - if(getToken()){ - user = await app.auth.asyncUser; - return user; - }else{ - return false; + // True when the logged-in user is a global admin (per user/me). Sync — only + // meaningful once isLoggedIn/forceLogin has resolved. + function isAdmin(){ + return !!(app.auth.perms && app.auth.perms.isAdmin); + } + + // Dual-mode: returns a Promise resolving to the user (or false), and calls + // an optional node-style callback with the same result. + function isLoggedIn(callback){ + var promise = loadUser().then(function(user){ + return user || false; + }); + + if(typeof callback === 'function'){ + promise.then(function(user){ + callback(null, user); + }, function(error){ + callback(error, false); + }); } + + return promise; } function logIn(args, callback){ @@ -252,62 +289,123 @@ app.auth = (function(app){ if(data.login){ setToken(data.token); } + loadUser(true); callback(error, !!data.token); }); } + // Clears the session only — the caller decides where to go next (the nav's + // Log Out button uses ui.logoutRedirect). function logOut(callback){ localStorage.removeItem('APIToken'); - location.replace(`/login${location.href.replace(location.origin, '')}`); - callback(); + userPromise = null; + app.auth.user = app.auth.perms = null; + if(typeof callback === 'function') callback(); } + // Constrain a redirect target to a same-origin absolute path. Rejects + // absolute URLs (open redirect), protocol-relative "//host" and "/\host", + // and non-path schemes like "javascript:" (XSS). Falls back to "/". + function safeInternalPath(path){ + if(typeof path !== 'string' || path.charAt(0) !== '/' + || path.charAt(1) === '/' || path.charAt(1) === '\\'){ + return '/'; + } + return path; + } + + // Consume an app token handed back by the OIDC callback via the URL + // fragment (#token=…&redirect=…). Stores it, strips the fragment, and + // forwards to the intended page. Returns true if a token was consumed. + function consumeTokenFragment(){ + if(!location.hash) return false; + var params = new URLSearchParams(location.hash.replace(/^#/, '')); + var token = params.get('token'); + if(!token) return false; + + setToken(token); + // redirect comes from the URL fragment (attacker-controllable); only + // allow a same-origin path so it can't become an open redirect / XSS. + var redirect = safeInternalPath(params.get('redirect') || '/'); + // Drop the token from the address bar before navigating on. + history.replaceState(null, '', location.pathname + location.search); + window.location.href = redirect; + return true; + } + + // Page-level gate. jQuery 4 removed $.holdReady, so an unauthenticated or + // unauthorised user is kept off the page by a redirect / an error panel + // rather than by pausing document ready. + // + // `requiredGroups` is a group CN or an OR-list of them; the synthetic + // `admin` group covers the OIDC clients' isAdmin flag. async function forceLogin(requiredGroups){ - $.holdReady(true); - if(!await app.auth.isLoggedIn()) app.auth.logOut(function(){}); + var user = await loadUser(); + + if(!user){ + logOut(function(){}); + location.replace('/login?redirect=' + encodeURIComponent( + location.pathname + location.search + )); + return false; + } if(user.onboardingRequired && location.pathname !== '/onboarding'){ location.replace('/onboarding'); + return false; } - if(requiredGroups){ - if(!await memberOf(requiredGroups)){ - console.log("Does not have permission!!!") - app.util.actionMessage( - `