1c52aaa92828b6ecc18dd5800fe6650b91daf696 max Fri Sep 18 07:22:10 2026 -0700 hgSession: the new Sessions page also lists the sessions saved on our other servers #Preview2 week - bugs introduced now will need a build patch to fix Every geo mirror node keeps its own namedSessionDb, so a session saved on genome-euro is invisible on genome.ucsc.edu and people do not find it again. The table now holds the sessions from every node, with a new Server column saying where each one lives, and the session name links to the server that holds it. Two new hgSession actions, both answered in main() before the cart is built, so that a request from another node leaves no cart, userDb or sessionDb row behind: hgS_doSessionListJson returns this server's sessions for the user named by the login cookie, as JSON. This is what the other nodes call. hgS_doMirrorSessions asks every other gbNode for that list and returns what came back. It sends all of the requests before reading any of the answers, so the wait is the slowest node rather than the sum of them. The fetch goes through our own server rather than from the browser to the other nodes, which keeps cross-origin requests out of it: apache handles the CORS headers differently on every node, and the two mirrors are administered by Bielefeld and RIKEN, while CGI code reaches them with the release. The user's login cookies, and only those, are passed on (wikiLinkLoginCookieHeader), which works because the nodes share a cookie salt. The page never waits for any of this: the table is drawn from this server's own sessions and the rows from the other servers are merged in as they arrive, keeping the current sort, search and page. A node that does not answer, times out, or runs a release without the endpoint is named in a line under the toolbar instead, and costs nothing else. Sessions from another server are shown but not managed here: no Share, Edit, Overwrite or Delete, no bulk-select checkbox, and the Update now shortcut keeps ignoring them. Table rows are keyed by a new uid rather than by the session name, since the same name can now be in the table twice, once per server. geoMirrorThisNode() and geoMirrorOtherNodes() read the node list from the hgcentral gbNode table, so nothing new has to be configured: browser.node already says which node a server is. diff --git src/hg/lib/geoMirror.c src/hg/lib/geoMirror.c index 3a905321e17..595a01b75ea 100644 --- src/hg/lib/geoMirror.c +++ src/hg/lib/geoMirror.c @@ -220,30 +220,72 @@ if ( (ip6AddrCmpBits(&ipStart, &ip) <= 0) && (ip6AddrCmpBits(&ipEnd , &ip) >= 0) ) { safef(response, sizeof response, "%s", row[2]); } } sqlFreeResult(&sr); return cloneString(response); } +static struct slPair *geoMirrorNodeList(boolean wantSelf) +/* Return gbNode as pairs of name=shortLabel, val=domain, ordered by node: either every node but + * this one (wantSelf FALSE) or only this one (wantSelf TRUE). */ +{ +if (!geoMirrorEnabled()) + return NULL; +char *myNode = geoMirrorNode(); +char *geoSuffix = cfgOptionDefault("browser.geoSuffix",""); +char query[256]; +sqlSafef(query, sizeof query, "SELECT node, domain, shortLabel from gbNode%s order by node", + geoSuffix); +struct sqlConnection *conn = hConnectCentral(); +struct sqlResult *sr = sqlGetResult(conn, query); +struct slPair *nodes = NULL; +char **row = NULL; +while ((row = sqlNextRow(sr)) != NULL) + { + if (sameString(row[0], myNode) != wantSelf) + continue; + slPairAdd(&nodes, row[2], cloneString(row[1])); + } +sqlFreeResult(&sr); +hDisconnectCentral(&conn); +slReverse(&nodes); +return nodes; +} + +struct slPair *geoMirrorThisNode() +/* Return this node (browser.node) as a single pair of name=shortLabel, val=domain, or NULL when + * geo mirroring is off or gbNode has no row for it. slPairFreeValsAndList when done. */ +{ +return geoMirrorNodeList(TRUE); +} + +struct slPair *geoMirrorOtherNodes() +/* Return the other geo mirror nodes, as pairs of name=shortLabel, val=domain, ordered by node. + * The node this CGI is running on (browser.node) is left out. Returns NULL when geo mirroring + * is off or this is the only node. slPairFreeValsAndList when done. */ +{ +return geoMirrorNodeList(FALSE); +} + char *geoMirrorMenu() /* Create customized geoMirror menu string for substitution of into * in htdocs/inc/globalNavBar.inc * Reads hgcentral geo tables and hg.conf settings. * Free the returned string when done. */ { struct dyString *dy = dyStringNew(0); // by default replacment is just an empty string. if (geoMirrorEnabled()) { dyStringAppend(dy, "