Kaladaran/briar
0
0
Fork 0
mirror of https://code.briarproject.org/briar/briar.git synced 2026-02-15 04:18:53 +01:00
Code Issues Projects Releases Wiki Activity

Poll contacts in series rather than parallel.

Browse Source
This commit is contained in:
akwizgran
2018-03-05 11:24:15 +00:00
parent 5025cf1e40
commit e9a6faad2d
1 changed files with 21 additions and 11 deletions
Download Patch File Download Diff File Expand all files Collapse all files

32
bramble-core/src/main/java/org/briarproject/bramble/plugin/bluetooth/BluetoothPlugin.java
View File

@@ -26,6 +26,9 @@ import org.briarproject.bramble.util.StringUtils;
import java.io.IOException; import java.io.IOException;
import java.security.SecureRandom; import java.security.SecureRandom;
import java.util.ArrayList;
import java.util.Collections;
import java.util.List;
import java.util.Map; import java.util.Map;
import java.util.Map.Entry; import java.util.Map.Entry;
import java.util.UUID; import java.util.UUID;
@@ -251,25 +254,32 @@ abstract class BluetoothPlugin<SS> implements DuplexPlugin, EventListener {
@Override @Override
public void poll(Map<ContactId, TransportProperties> contacts) { public void poll(Map<ContactId, TransportProperties> contacts) {
if (!isRunning() || !shouldAllowContactConnections()) return; if (!isRunning() || !shouldAllowContactConnections()) return;
backoff.increment(); // Try to connect to known devices in a random order
// Try to connect to known devices in parallel List<Entry<ContactId, TransportProperties>> entries = new ArrayList<>();
for (Entry<ContactId, TransportProperties> e : contacts.entrySet()) { for (Entry<ContactId, TransportProperties> e : contacts.entrySet()) {
String address = e.getValue().get(PROP_ADDRESS); TransportProperties p = e.getValue();
if (StringUtils.isNullOrEmpty(address)) continue; if (StringUtils.isNullOrEmpty(p.get(PROP_ADDRESS))) continue;
String uuid = e.getValue().get(PROP_UUID); if (StringUtils.isNullOrEmpty(p.get(PROP_UUID))) continue;
if (StringUtils.isNullOrEmpty(uuid)) continue; entries.add(e);
ContactId c = e.getKey(); }
ioExecutor.execute(() -> { if (entries.isEmpty()) return;
Collections.shuffle(entries);
backoff.increment();
ioExecutor.execute(() -> {
for (Entry<ContactId, TransportProperties> e : entries) {
if (!isRunning() || !shouldAllowContactConnections()) return; if (!isRunning() || !shouldAllowContactConnections()) return;
if (!connectionLimiter.canOpenContactConnection()) return; if (!connectionLimiter.canOpenContactConnection()) return;
TransportProperties p = e.getValue();
String address = p.get(PROP_ADDRESS);
String uuid = p.get(PROP_UUID);
DuplexTransportConnection conn = connect(address, uuid); DuplexTransportConnection conn = connect(address, uuid);
if (conn != null) { if (conn != null) {
backoff.reset(); backoff.reset();
if (connectionLimiter.contactConnectionOpened(conn)) if (connectionLimiter.contactConnectionOpened(conn))
callback.outgoingConnectionCreated(c, conn); callback.outgoingConnectionCreated(e.getKey(), conn);
} }
}); }
} });
} }
@Nullable @Nullable