From 84e040605b07fefc5ca9d6d1560d74d14123cbd5 Mon Sep 17 00:00:00 2001 From: akwizgran Date: Wed, 7 Mar 2018 16:47:08 +0000 Subject: [PATCH] Don't reuse the same ConnectionChooser every time. This is a fix for a backporting mistake. --- .../keyagreement/KeyAgreementTaskFactoryImpl.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/bramble-core/src/main/java/org/briarproject/bramble/keyagreement/KeyAgreementTaskFactoryImpl.java b/bramble-core/src/main/java/org/briarproject/bramble/keyagreement/KeyAgreementTaskFactoryImpl.java index 08a03b049..d186dcc28 100644 --- a/bramble-core/src/main/java/org/briarproject/bramble/keyagreement/KeyAgreementTaskFactoryImpl.java +++ b/bramble-core/src/main/java/org/briarproject/bramble/keyagreement/KeyAgreementTaskFactoryImpl.java @@ -10,6 +10,7 @@ import org.briarproject.bramble.api.plugin.PluginManager; import javax.annotation.concurrent.Immutable; import javax.inject.Inject; +import javax.inject.Provider; @Immutable @NotNullByDefault @@ -19,22 +20,22 @@ class KeyAgreementTaskFactoryImpl implements KeyAgreementTaskFactory { private final EventBus eventBus; private final PayloadEncoder payloadEncoder; private final PluginManager pluginManager; - private final ConnectionChooser connectionChooser; + private final Provider connectionChooserProvider; @Inject KeyAgreementTaskFactoryImpl(CryptoComponent crypto, EventBus eventBus, PayloadEncoder payloadEncoder, PluginManager pluginManager, - ConnectionChooser connectionChooser) { + Provider connectionChooserProvider) { this.crypto = crypto; this.eventBus = eventBus; this.payloadEncoder = payloadEncoder; this.pluginManager = pluginManager; - this.connectionChooser = connectionChooser; + this.connectionChooserProvider = connectionChooserProvider; } @Override public KeyAgreementTask createTask() { return new KeyAgreementTaskImpl(crypto, eventBus, payloadEncoder, - pluginManager, connectionChooser); + pluginManager, connectionChooserProvider.get()); } }