Skip to content
Snippets Groups Projects
Verified Commit 5cdb35b8 authored by Julian Dehm's avatar Julian Dehm
Browse files

Send empty transport properties to the mailbox

parent 5b17fe2d
No related branches found
No related tags found
No related merge requests found
...@@ -40,6 +40,12 @@ public interface TransportPropertyManager { ...@@ -40,6 +40,12 @@ public interface TransportPropertyManager {
Map<TransportId, TransportProperties> getLocalProperties() Map<TransportId, TransportProperties> getLocalProperties()
throws DbException; throws DbException;
/**
* Returns the local anonymized transport properties for all transports.
*/
Map<TransportId, TransportProperties> getLocalAnonymizedProperties()
throws DbException;
/** /**
* Returns the local transport properties for all transports. * Returns the local transport properties for all transports.
* <p/> * <p/>
......
...@@ -47,6 +47,7 @@ import javax.inject.Inject; ...@@ -47,6 +47,7 @@ import javax.inject.Inject;
import static java.util.logging.Level.WARNING; import static java.util.logging.Level.WARNING;
import static org.briarproject.bramble.api.contact.ContactType.CONTACT; import static org.briarproject.bramble.api.contact.ContactType.CONTACT;
import static org.briarproject.bramble.api.contact.ContactType.MAILBOX_OWNER;
import static org.briarproject.bramble.api.contact.RecordTypes.CONTACT_INFO; import static org.briarproject.bramble.api.contact.RecordTypes.CONTACT_INFO;
import static org.briarproject.bramble.api.identity.AuthorConstants.MAX_SIGNATURE_LENGTH; import static org.briarproject.bramble.api.identity.AuthorConstants.MAX_SIGNATURE_LENGTH;
import static org.briarproject.bramble.util.LogUtils.logException; import static org.briarproject.bramble.util.LogUtils.logException;
...@@ -150,7 +151,11 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask { ...@@ -150,7 +151,11 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask {
// Get the local transport properties // Get the local transport properties
Map<TransportId, TransportProperties> localProperties; Map<TransportId, TransportProperties> localProperties;
try { try {
localProperties = transportPropertyManager.getLocalProperties(); if (localType == MAILBOX_OWNER)
localProperties =
transportPropertyManager.getLocalAnonymizedProperties();
else
localProperties = transportPropertyManager.getLocalProperties();
} catch (DbException e) { } catch (DbException e) {
logException(LOG, WARNING, e); logException(LOG, WARNING, e);
listener.contactExchangeFailed(); listener.contactExchangeFailed();
...@@ -332,8 +337,6 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask { ...@@ -332,8 +337,6 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask {
contactId = contactManager.addContact(txn, remoteAuthor, contactId = contactManager.addContact(txn, remoteAuthor,
localAuthor.getId(), masterSecret, timestamp, alice, localAuthor.getId(), masterSecret, timestamp, alice,
true, true); true, true);
transportPropertyManager.addRemoteProperties(txn, contactId,
remoteProperties);
break; break;
case MAILBOX_OWNER: case MAILBOX_OWNER:
contactId = contactId =
...@@ -346,12 +349,12 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask { ...@@ -346,12 +349,12 @@ class ContactExchangeTaskImpl extends Thread implements ContactExchangeTask {
contactManager.addPrivateMailbox(txn, remoteAuthor, contactManager.addPrivateMailbox(txn, remoteAuthor,
localAuthor.getId(), masterSecret, localAuthor.getId(), masterSecret,
timestamp, alice); timestamp, alice);
transportPropertyManager.addRemoteProperties(txn, contactId,
remoteProperties);
break; break;
default: default:
throw new RuntimeException("Invalid remote contact type"); throw new RuntimeException("Invalid remote contact type");
} }
transportPropertyManager.addRemoteProperties(txn, contactId,
remoteProperties);
db.commitTransaction(txn); db.commitTransaction(txn);
} finally { } finally {
......
...@@ -32,6 +32,7 @@ import org.briarproject.bramble.api.versioning.ClientVersioningManager.ClientVer ...@@ -32,6 +32,7 @@ import org.briarproject.bramble.api.versioning.ClientVersioningManager.ClientVer
import java.util.Arrays; import java.util.Arrays;
import java.util.Collection; import java.util.Collection;
import java.util.Collections;
import java.util.HashMap; import java.util.HashMap;
import java.util.Map; import java.util.Map;
import java.util.Map.Entry; import java.util.Map.Entry;
...@@ -147,6 +148,16 @@ class TransportPropertyManagerImpl implements TransportPropertyManager, ...@@ -147,6 +148,16 @@ class TransportPropertyManagerImpl implements TransportPropertyManager,
} }
} }
@Override
public Map<TransportId, TransportProperties> getLocalAnonymizedProperties()
throws DbException {
Map<TransportId, TransportProperties> properties = getLocalProperties();
for (Entry e : properties.entrySet()) {
e.setValue(Collections.EMPTY_MAP);
}
return properties;
}
@Override @Override
public Map<TransportId, TransportProperties> getLocalProperties() public Map<TransportId, TransportProperties> getLocalProperties()
throws DbException { throws DbException {
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment