aca9143822a9c12b16b33d479a5abb6495b6b4f0
braney
  Wed Aug 19 06:59:39 2026 -0700
lib, hgc, hgTables, hgHubConnect: share hubEncode, refs #38123

hgc.c and hgTables.c each had their own copy of hubEncode().  The two
bodies were identical, so move the function to hubConnect.c next to
isHubTrack() and let both CGIs call the one definition.

Also use cgiEncode() rather than htmlEncode() for the db= value in the
hgHubConnect assembly list.  That value goes in a query string, so
cgiEncode() is the right function there.  Genome names are already
limited to a safe character set, so this changes no output today.

diff --git src/hg/hgHubConnect/hgHubConnect.c src/hg/hgHubConnect/hgHubConnect.c
index fd56a944ed5..16b15f03ab9 100644
--- src/hg/hgHubConnect/hgHubConnect.c
+++ src/hg/hgHubConnect/hgHubConnect.c
@@ -1464,31 +1464,31 @@
 hPrintf("<P><B>Assemblies:</B> Select an assembly below to start browsing the tracks of this hub:<P>");
 
 hPrintf("<ul>");
 
 for(; genomeList; genomeList = genomeList->next)
     {
     char *desc = genomeList->description;
     if (desc==NULL)
         desc = hFreezeFromDb(genomeList->name);
 
     char *org = trackHubSkipHubName( genomeList->organism );
     if (org==NULL)
         org = trackHubSkipHubName(hOrganism(genomeList->name));
 
     hPrintf("<li>Open: <A href=\"../cgi-bin/hgTracks?db=%s&%s&position=lastDbPos\">%s: %s</A></li>",
-        htmlEncode(genomeList->name), cartSidUrlString(cart), htmlEncode(org), htmlEncode(desc));
+        cgiEncode(genomeList->name), cartSidUrlString(cart), htmlEncode(org), htmlEncode(desc));
     }
 hPrintf("</ul>");
 
 hPrintf("<P><A href=\"./hgHubConnect#unlistedHubs\">Connect another hub</A> to add more tracks.</P>");
 
 hPrintf("<P><I>Hubs are created and maintained by external sources. UCSC is not responsible for their content and status.</I></P>");
 
 return 1;
 }
 
 static void doResetHub(struct cart *theCart)
 {
 char *url = cartOptionalString(cart, hgHubCheckUrl);
 
 if (url != NULL)