0b638d6d33466fc848c63d261eab155a22741bdf gperez2 Wed Sep 2 16:58:32 2026 -0700 Changing hgLogin's "choose a username" page back to the original generic text for ORCID and CILogon sign-ins, per code review feedback on #38213 (the provider-name check was unreliable and hardcoded the mirror name), refs #37929 diff --git src/hg/hgLogin/hgLogin.c src/hg/hgLogin/hgLogin.c index 85904827b20..b570e879194 100644 --- src/hg/hgLogin/hgLogin.c +++ src/hg/hgLogin/hgLogin.c @@ -1927,44 +1927,30 @@ if (isEmpty(provider) || !pendingIdentityValid()) { clearPendingIdentity(); displayLoginPage(conn); return; } char *suggested = cartUsualString(cart, "hgLogin_userName", ""); if (isEmpty(suggested)) suggested = suggestUsername(conn, email, name); char *encSuggested = htmlEncode(suggested); // both go into value="" attributes; escape (XSS) char *encEmail = htmlEncode(email); hPrintf("
A new genome browser account is created for any ORCID sign-in not seen before, " - "because ORCID only shares an ORCID iD, not an email address. So you cannot sign into an " - "existing account using ORCID. Use another sign-in option instead if you don't want to " - "create a new account.
" - "Pick a username for your %s account. You can change the suggested name below.
", - brwName); -else if (sameWord(provider, "cilogon")) - hPrintf("A new genome browser account is created for any CILogon sign-in not seen before. " - "So you cannot sign into an existing account using CILogon this way. Use another sign-in " - "option instead if you don't want to create a new account.
" - "Pick a username for your %s account. You can change the suggested name below.
", - brwName); -else hPrintf("You signed in with %s. Pick a username for your new %s account. " "You can change the suggested name below.
", oauthProviderLabel(provider), brwName); printUsernameNote(); hPrintf("%s", errMsg ? errMsg : ""); hPrintf("