32cd2100d9dd11140705c848850e86c4f79191c4 max Tue Sep 1 07:02:04 2026 -0700 Reusable "view snapshot" sessions for durable, minimal Share-a-link links "Share a link" could only share the whole cart: it saved a session holding every track, setting and position, which bloated the central db and leaked the sharer's unrelated tracks to whoever opened the link. Anonymous share sessions (under the reserved user "l") were also never reaped, so they accumulated forever, and their 8-char names were generated client-side with no uniqueness check, so two shares could collide and silently overwrite. Adds a lightweight "snapshot session" facility (lib/snapshotSession.c): a snapshot stores only the handful of cart variables a feature declares (a registered snapshotType, e.g. "blat" -> {db, blatLastBigBed}), moves only those variables' trash files into durable sessionData storage, and is saved under a "__"-prefixed name. The "__" marks it machine-made: hidden from the My Sessions list by default and eligible for reaping. For the anonymous "l" owner the durable files fan out over two extra hash levels so one directory never fills with millions of entries. Every anonymous link now shares one server-side name generator (snapshotNewName): a unique (db-checked), crypto-strong, "__"-prefixed token, so tokens never collide. The hgSession doSaveSessionJson endpoint gained hgS_snapshotType (save a minimal snapshot rather than the whole cart) and hgS_doAnonName (reserve a unique anonymous name without saving, so the top-right dialog can preview the exact link before it is created). Wired three callers to the facility: - hgc htcBlatAlign "Share a link": a minimal "blat" snapshot instead of a full-cart anonymous session. - hgBlat results "Share a link": creates a "blat" snapshot on click and reveals its ?u=&s= reopen link (rebuilt from the durable bigPsl by the existing doShareReopen), replacing the trash-only reveal. - top-right "Share a link": anonymous links use the reserved server name; logged-in named shares are unchanged. snapshotReaper (hg/utils) garbage-collects abandoned anonymous snapshots: it deletes user "l" "__" rows whose lastUse is older than the TTL (hg.conf snapshot.ttlDays, default ~4 years) and removes their durable files. lastUse is bumped on every open by the existing session load, so a link stays alive as long as it is used. Meant to run from the trash-cleaner cron. Also folds in the recent Share-dialog work in these files: the auto share name is a short "_" prefix instead of "share_", the dialog previews the link and creates it only when the button is clicked (no orphan session just from opening the dialog), an optional name field with an overwrite warning, and the anonymous save path forces the reap-eligible "__" name. refs #38197 diff --git src/hg/js/hgSession.js src/hg/js/hgSession.js index a1dda41235a..1cba1a3abdd 100644 --- src/hg/js/hgSession.js +++ src/hg/js/hgSession.js @@ -35,39 +35,40 @@ var sessData = null; // set in sessionBuild: {config, sessions} var sessDt = null; // the DataTable API var sessSelectMode = false; // bulk-select (checkbox column) shown? function sessEnc(s) { // HTML-escape via the shared utils.js helper (escapes quotes too, so it is attribute-safe). return (typeof htmlEncode === 'function') ? htmlEncode(String(s == null ? '' : s)) : String(s); } function sessNum(n) { return (typeof commify === 'function') ? commify(n) : String(n); } function sessRandomShareName() { - // Mirror the server's auto/anonymous share-name convention ("share_" + 8 URL-safe alphanumeric - // chars). hgSession.c's doSaveSessionJson generates the same style server-side for the top-right - // "Share a link"; we generate it here so the confirm dialog can show the name before saving. + // Auto-name convention for machine-generated session names: a leading "_" (which marks the name + // as internally generated, and is kept verbatim by the short-link encoder) followed by 8 URL-safe + // alphanumeric chars. This is the single source of the convention: the top-right "Share a link" + // menu (topLinks.js) and hgSession.c both defer to a name generated here. var chars = 'ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz0123456789'; var s = ''; for (var i = 0; i < 8; i++) { s += chars.charAt(Math.floor(Math.random() * chars.length)); } - return 'share_' + s; + return '_' + s; } function sessMbpPos(pos) { // Shorten a position to megabases to save horizontal space: // "chr7:155799529-155812871" -> "chr7:155.80-155.81 Mbp". Falls back to the raw string // if it doesn't parse. var m = /^(.+):([0-9,]+)-([0-9,]+)$/.exec(String(pos)); if (!m) { return String(pos); } var s = parseInt(m[2].replace(/,/g, ''), 10); var e = parseInt(m[3].replace(/,/g, ''), 10); if (isNaN(s) || isNaN(e)) { return String(pos); } return m[1] + ':' + (s / 1e6).toFixed(2) + '-' + (e / 1e6).toFixed(2) + ' Mbp'; } function sessMsg(text, cls) { @@ -420,31 +421,31 @@ } // Build the base params for an action on a given session (decoded name; the CGI re-encodes it). function sessActParams(action, row) { var p = {}; p[action] = '1'; p[SESS_P.oldName] = row.name; return p; } // ---- Save current view --------------------------------------------------- function sessDoSave() { var name = document.getElementById('sessSaveName').value.trim(); if (!name) { - // Empty name: offer to save under a server-style random "share_XXXXXXXX" name, after + // Empty name: offer to save under a random internal "_XXXXXXXX" name, after // confirming the user really meant to leave it blank. var rand = sessRandomShareName(); sessConfirm({ title: 'Save without a name?', bodyHtml: 'You left the session name empty. Your session will be saved under the ' + 'randomly generated name ' + sessEnc(rand) + '.

You can also create ' + 'these quick share links any time from the Share a link option at the top ' + 'right of every Genome Browser page.', okLabel: 'Save session', onOk: function() { sessModalClose(); sessDoSaveWithName(rand); } }); return; } sessDoSaveWithName(name); }