From 55d5a97d99ea3fb7ae351b229f3d319de0b6628b Mon Sep 17 00:00:00 2001 From: Craig Raw Date: Tue, 30 Jan 2024 08:03:38 +0200 Subject: [PATCH] avoid saving certificates for public servers --- .../sparrowwallet/sparrow/net/TcpOverTlsTransport.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/src/main/java/com/sparrowwallet/sparrow/net/TcpOverTlsTransport.java b/src/main/java/com/sparrowwallet/sparrow/net/TcpOverTlsTransport.java index df34b435..4da5d045 100644 --- a/src/main/java/com/sparrowwallet/sparrow/net/TcpOverTlsTransport.java +++ b/src/main/java/com/sparrowwallet/sparrow/net/TcpOverTlsTransport.java @@ -117,10 +117,11 @@ public class TcpOverTlsTransport extends TcpTransport { } protected boolean shouldSaveCertificate() { - //Avoid saving the certificates for blockstream.info public servers - they change too often and encourage approval complacency - if(PublicElectrumServer.BLOCKSTREAM_INFO.getServer().getHost().equals(server.getHost()) - || PublicElectrumServer.ELECTRUM_BLOCKSTREAM_INFO.getServer().getHost().equals(server.getHost())) { - return false; + //Avoid saving the certificates for public servers - they change often, encourage approval complacency, and there is little a user can do to check + for(PublicElectrumServer publicElectrumServer : PublicElectrumServer.getServers()) { + if(publicElectrumServer.getServer().getHost().equals(server.getHost())) { + return false; + } } return Storage.getCertificateFile(server.getHost()) == null;