Skip to content

Commit

Permalink
fix: Removed pointless certificate caching and fixed an SonarCloud ip…
Browse files Browse the repository at this point in the history 
…s error

Signed-off-by: ivaylogarnev-limechain <[email protected]>
  • Loading branch information
@ivaylogarnev-limechain
ivaylogarnev-limechain committed Dec 16, 2024
1 parent 8fa1ab7 commit 8147866
Show file tree
Hide file tree
Showing 2 changed files with 101 additions and 111 deletions.
14 changes: 2 additions & 12 deletions src/channel/NodeChannel.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -25,8 +25,6 @@ import GrpcServicesError from "../grpc/GrpcServiceError.js";
import GrpcStatus from "../grpc/GrpcStatus.js";
import { ALL_NETWORK_IPS } from "../constants/ClientConstants.js";

/** @type {{ [key: string]: string }} */
const certificateCache = {};
/** @type {{ [key: string]: Client }} */
const clientCache = {};

Expand Down Expand Up @@ -82,10 +80,6 @@ export default class NodeChannel extends Channel {
* @returns {Promise<string>}
*/
async _retrieveCertificate(address) {

Check warning on line 82 in src/channel/NodeChannel.js

View workflow job for this annotation

GitHub Actions / Test using Node 16 

'address' is defined but never used

Check warning on line 82 in src/channel/NodeChannel.js

View workflow job for this annotation

GitHub Actions / Integration Tests on Node 16 

'address' is defined but never used

Check warning on line 82 in src/channel/NodeChannel.js

View workflow job for this annotation

GitHub Actions / Build using Node 16 

'address' is defined but never used

Check warning on line 82 in src/channel/NodeChannel.js

View workflow job for this annotation

GitHub Actions / Integration Tests on Node 18 

'address' is defined but never used

Check warning on line 82 in src/channel/NodeChannel.js

View workflow job for this annotation

GitHub Actions / Build using Node 18 

'address' is defined but never used
if (certificateCache[address]) {
return certificateCache[address];
}

return new Promise((resolve, reject) => {
const socket = tls.connect(
{
Expand All @@ -98,10 +92,7 @@ export default class NodeChannel extends Channel {
const cert = socket.getPeerCertificate();

if (cert && cert.raw) {
const certPEM = this.bytesToPem(cert.raw);
certificateCache[address] = certPEM;

resolve(certPEM);
resolve(this.bytesToPem(cert.raw));
} else {
reject(new Error("No certificate retrieved."));
}
Expand Down Expand Up @@ -162,7 +153,6 @@ export default class NodeChannel extends Channel {
if (this._client) {
this._client.close();
delete clientCache[this.address];
delete certificateCache[this.address];
}
}

Expand All @@ -189,7 +179,7 @@ export default class NodeChannel extends Channel {
callback(
new GrpcServicesError(
GrpcStatus.Timeout,
ALL_NETWORK_IPS[this.nodeIp],
ALL_NETWORK_IPS[`${this.nodeIp}:`],
),
);
} else {
Expand Down
198 changes: 99 additions & 99 deletions src/constants/ClientConstants.js
View file
Original file line number Diff line number Diff line change
Expand Up @@ -80,106 +80,106 @@ export const ALL_WEB_NETWORK_NODES = {
*/
export const ALL_NETWORK_IPS = {
// MAINNET
"34.239.82.6": "0.0.3",
"35.237.200.180": "0.0.3",
"3.130.52.236": "0.0.4",
"35.186.191.247": "0.0.4",
"3.18.18.254": "0.0.5",
"35.192.2.25": "0.0.5",
"74.50.117.35": "0.0.5",
"23.111.186.250": "0.0.5",
"107.155.64.98": "0.0.5",
"13.52.108.243": "0.0.6",
"35.199.161.108": "0.0.6",
"3.114.54.4": "0.0.7",
"35.203.82.240": "0.0.7",
"35.236.5.219": "0.0.8",
"35.183.66.150": "0.0.8",
"35.181.158.250": "0.0.9",
"35.197.192.225": "0.0.9",
"177.154.62.234": "0.0.10",
"3.248.27.48": "0.0.10",
"35.242.233.154": "0.0.10",
"13.53.119.185": "0.0.11",
"35.240.118.96": "0.0.11",
"35.204.86.32": "0.0.12",
"35.177.162.180": "0.0.12",
"34.215.192.104": "0.0.13",
"35.234.132.107": "0.0.13",
"52.8.21.141": "0.0.14",
"35.236.2.27": "0.0.14",
"35.228.11.53": "0.0.15",
"3.121.238.26": "0.0.15",
"34.91.181.183": "0.0.16",
"18.157.223.230": "0.0.16",
"34.86.212.247": "0.0.17",
"18.232.251.19": "0.0.17",
"141.94.175.187": "0.0.18",
"34.89.87.138": "0.0.19",
"18.168.4.59": "0.0.19",
"34.82.78.255": "0.0.20",
"52.39.162.216": "0.0.20",
"34.76.140.109": "0.0.21",
"13.36.123.209": "0.0.21",
"52.78.202.34": "0.0.22",
"34.64.141.166": "0.0.22",
"3.18.91.176": "0.0.23",
"35.232.244.145": "0.0.23",
"69.167.169.208": "0.0.23",
"34.89.103.38": "0.0.24",
"18.135.7.211": "0.0.24",
"34.93.112.7": "0.0.25",
"13.232.240.207": "0.0.25",
"13.228.103.14": "0.0.26",
"34.87.150.174": "0.0.26",
"13.56.4.96": "0.0.27",
"34.125.200.96": "0.0.27",
"35.198.220.75": "0.0.28",
"18.139.47.5": "0.0.28",
"54.74.60.120": "0.0.29",
"34.142.71.129": "0.0.29",
"80.85.70.197": "0.0.29",
"35.234.249.150": "0.0.30",
"34.201.177.212": "0.0.30",
"217.76.57.165": "0.0.31",
"3.77.94.254": "0.0.31",
"34.107.78.179": "0.0.31",
"34.86.186.151": "0.0.32",
"3.20.81.230": "0.0.32",
"18.136.65.22": "0.0.33",
"34.142.172.228": "0.0.33",
"34.16.139.248": "0.0.34",
"35.155.212.90": "0.0.34",
"34.239.82.6:": "0.0.3",
"35.237.200.180:": "0.0.3",
"3.130.52.236:": "0.0.4",
"35.186.191.247:": "0.0.4",
"3.18.18.254:": "0.0.5",
"35.192.2.25:": "0.0.5",
"74.50.117.35:": "0.0.5",
"23.111.186.250:": "0.0.5",
"107.155.64.98:": "0.0.5",
"13.52.108.243:": "0.0.6",
"35.199.161.108:": "0.0.6",
"3.114.54.4:": "0.0.7",
"35.203.82.240:": "0.0.7",
"35.236.5.219:": "0.0.8",
"35.183.66.150:": "0.0.8",
"35.181.158.250:": "0.0.9",
"35.197.192.225:": "0.0.9",
"177.154.62.234:": "0.0.10",
"3.248.27.48:": "0.0.10",
"35.242.233.154:": "0.0.10",
"13.53.119.185:": "0.0.11",
"35.240.118.96:": "0.0.11",
"35.204.86.32:": "0.0.12",
"35.177.162.180:": "0.0.12",
"34.215.192.104:": "0.0.13",
"35.234.132.107:": "0.0.13",
"52.8.21.141:": "0.0.14",
"35.236.2.27:": "0.0.14",
"35.228.11.53:": "0.0.15",
"3.121.238.26:": "0.0.15",
"34.91.181.183:": "0.0.16",
"18.157.223.230:": "0.0.16",
"34.86.212.247:": "0.0.17",
"18.232.251.19:": "0.0.17",
"141.94.175.187:": "0.0.18",
"34.89.87.138:": "0.0.19",
"18.168.4.59:": "0.0.19",
"34.82.78.255:": "0.0.20",
"52.39.162.216:": "0.0.20",
"34.76.140.109:": "0.0.21",
"13.36.123.209:": "0.0.21",
"52.78.202.34:": "0.0.22",
"34.64.141.166:": "0.0.22",
"3.18.91.176:": "0.0.23",
"35.232.244.145:": "0.0.23",
"69.167.169.208:": "0.0.23",
"34.89.103.38:": "0.0.24",
"18.135.7.211:": "0.0.24",
"34.93.112.7:": "0.0.25",
"13.232.240.207:": "0.0.25",
"13.228.103.14:": "0.0.26",
"34.87.150.174:": "0.0.26",
"13.56.4.96:": "0.0.27",
"34.125.200.96:": "0.0.27",
"35.198.220.75:": "0.0.28",
"18.139.47.5:": "0.0.28",
"54.74.60.120:": "0.0.29",
"34.142.71.129:": "0.0.29",
"80.85.70.197:": "0.0.29",
"35.234.249.150:": "0.0.30",
"34.201.177.212:": "0.0.30",
"217.76.57.165:": "0.0.31",
"3.77.94.254:": "0.0.31",
"34.107.78.179:": "0.0.31",
"34.86.186.151:": "0.0.32",
"3.20.81.230:": "0.0.32",
"18.136.65.22:": "0.0.33",
"34.142.172.228:": "0.0.33",
"34.16.139.248:": "0.0.34",
"35.155.212.90:": "0.0.34",
// TESTNET
"34.94.106.61": "0.0.3",
"50.18.132.211": "0.0.3",
"3.212.6.13": "0.0.4",
"35.237.119.55": "0.0.4",
"35.245.27.193": "0.0.5",
"52.20.18.86": "0.0.5",
"34.83.112.116": "0.0.6",
"54.70.192.33": "0.0.6",
"34.94.160.4": "0.0.7",
"54.176.199.109": "0.0.7",
"35.155.49.147": "0.0.8",
"34.106.102.218": "0.0.8",
"34.133.197.230": "0.0.9",
"52.14.252.207": "0.0.9",
"34.94.106.61:": "0.0.3",
"50.18.132.211:": "0.0.3",
"3.212.6.13:": "0.0.4",
"35.237.119.55:": "0.0.4",
"35.245.27.193:": "0.0.5",
"52.20.18.86:": "0.0.5",
"34.83.112.116:": "0.0.6",
"54.70.192.33:": "0.0.6",
"34.94.160.4:": "0.0.7",
"54.176.199.109:": "0.0.7",
"35.155.49.147:": "0.0.8",
"34.106.102.218:": "0.0.8",
"34.133.197.230:": "0.0.9",
"52.14.252.207:": "0.0.9",
// LOCAL NODE
"127.0.0.1": "0.0.3",
"127.0.0.1:": "0.0.3",
// PREVIEW NET
"3.211.248.172": "0.0.3",
"35.231.208.148": "0.0.3",
"35.199.15.177": "0.0.4",
"3.133.213.146": "0.0.4",
"35.225.201.195": "0.0.5",
"52.15.105.130": "0.0.5",
"54.241.38.1": "0.0.6",
"35.247.109.135": "0.0.6",
"54.177.51.127": "0.0.7",
"35.235.65.51": "0.0.7",
"34.106.247.65": "0.0.8",
"35.83.89.171": "0.0.8",
"50.18.17.93": "0.0.9",
"34.125.23.49": "0.0.9",
"3.211.248.172:": "0.0.3",
"35.231.208.148:": "0.0.3",
"35.199.15.177:": "0.0.4",
"3.133.213.146:": "0.0.4",
"35.225.201.195:": "0.0.5",
"52.15.105.130:": "0.0.5",
"54.241.38.1:": "0.0.6",
"35.247.109.135:": "0.0.6",
"54.177.51.127:": "0.0.7",
"35.235.65.51:": "0.0.7",
"34.106.247.65:": "0.0.8",
"35.83.89.171:": "0.0.8",
"50.18.17.93:": "0.0.9",
"34.125.23.49:": "0.0.9",
};

0 comments on commit 8147866

Please sign in to comment.