disable getVersion to snodes calls as we don't need it anymore

js/modules/debuglogs.js

@@ -10,6 +10,7 @@ const USER_AGENT = `Session ${VERSION}`;
 
 //      upload :: String -> Promise URL
 exports.upload = async content => {
+  window.log.warn('insecureNodeFetch => upload debugLogs');
   const signedForm = await insecureNodeFetch(BASE_URL, {
     headers: {
       'user-agent': USER_AGENT,

ts/session/snode_api/lokiRpc.ts

@@ -34,12 +34,9 @@ async function lokiFetch(
     if (url.match(/https:\/\//)) {
       // import that this does not get set in lokiFetch fetchOptions
       fetchOptions.agent = snodeHttpsAgent;
-    } else {
-      window.log.warn(
-        'lokirpc:::lokiFetch - insecureNodeFetch http communication',
-        url
-      );
     }
+    window.log.warn(`insecureNodeFetch => lokiFetch of ${url}`);
+
     const response = await insecureNodeFetch(url, fetchOptions);
 
     if (!response.ok) {

ts/session/snode_api/serviceNodeAPI.ts

@@ -19,6 +19,10 @@ import {
   updateSnodesFor,
 } from './snodePool';
 
+/**
+ * Currently unused. If we need it again, be sure to update it to onion routing rather
+ * than using a plain nodeFetch
+ */
 export async function getVersion(
   node: Snode,
   retries: number = 0
@@ -28,8 +32,7 @@ export async function getVersion(
   const { log } = window;
 
   try {
-    // window.log.info('insecureNodeFetch => plaintext for getVersion');
+    window.log.warn('insecureNodeFetch => plaintext for getVersion');
-
     const result = await insecureNodeFetch(
       `https://${node.ip}:${node.port}/get_stats/v1`,
       {

ts/session/snode_api/snodePool.ts

@@ -136,13 +136,18 @@ function compareSnodes(lhs: any, rhs: any): boolean {
   return lhs.pubkey_ed25519 === rhs.pubkey_ed25519;
 }
 
-// WARNING: this leaks our IP to all snodes but with no other identifying information
+/**
-// except "that a client started up" or "ran out of random pool snodes"
+ * Request the version of the snode.
-// and the order of the list is randomized, so a snode can't tell if it just started or not
+ * THIS IS AN INSECURE NODE FETCH and leaks our IP to all snodes but with no other identifying information
+ * except "that a client started up" or "ran out of random pool snodes"
+ * and the order of the list is randomized, so a snode can't tell if it just started or not
+ */
 async function requestVersion(node: any): Promise<void> {
   const { log } = window;
 
-  const result = await getVersion(node);
+  // WARNING: getVersion is doing an insecure node fetch.
+  // be sure to update getVersion to onion routing if we need this call again.
+  const result = false; // await getVersion(node);
 
   if (result === false) {
     return;
@@ -177,8 +182,13 @@ export function getNodesMinVersion(minVersion: string): Array<Snode> {
   );
 }
 
-// now get version for all snodes
+/**
-// also acts an early online test/purge of bad nodes
+ * Currently unused as it makes call over insecure node fetch and we don't need
+ * to filter out nodes by versions anymore.
+ *
+ * now get version for all snodes
+ * also acts an early online test/purge of bad nodes
+ */
 export async function getAllVersionsForRandomSnodePool(): Promise<void> {
   const { log } = window;
 
@@ -284,7 +294,9 @@ async function refreshRandomPoolDetail(seedNodes: Array<any>): Promise<void> {
       randomSnodePool.length,
       'snodes'
     );
-    void getAllVersionsForRandomSnodePool();
+    // Warning: the call below will call getVersions to all existing nodes.
+    // And not with onion routing
+    // void getAllVersionsForRandomSnodePool();
   } catch (e) {
     log.warn('LokiSnodeAPI::refreshRandomPool - error', e.code, e.message);
     /*

ts/session/snode_api/swarmPollingStub.ts

@@ -12,6 +12,7 @@ export class SwarmPollingStub extends SwarmPolling {
       method: 'GET',
     };
 
+    // insecureNodeFetch but this is a stub
     const res = await insecureNodeFetch(
       `${this.baseUrl}/messages?pubkey=${pubkeyStr}`,
       get

ts/test/session/integration/stubs/stub_message_api.ts

@@ -23,6 +23,8 @@ class StubMessageAPI {
     };
 
     const data64 = StringUtils.decode(data, 'base64');
+    // insecureNodeFetch but this is a stub
+
     await insecureNodeFetch(
       `${
         this.baseUrl