Skip to content

Commit 7e95ea7

Browse files
committed
Log identifier of failed profile download
Fixes #2003
1 parent 2991cda commit 7e95ea7

1 file changed

Lines changed: 9 additions & 3 deletions

File tree

lib/src/main/java/org/asamk/signal/manager/helper/ProfileHelper.java

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -134,7 +134,9 @@ public ExpiringProfileKeyCredential getExpiringProfileKeyCredential(RecipientId
134134
SignalServiceProfile.RequestType.PROFILE_AND_CREDENTIAL,
135135
false));
136136
} catch (IOException e) {
137-
logger.warn("Failed to retrieve profile key credential, ignoring: {}", e.getMessage());
137+
logger.warn("Failed to retrieve profile key credential for {}, ignoring: {}",
138+
context.getRecipientHelper().resolveSignalServiceAddress(recipientId).getIdentifier(),
139+
e.getMessage());
138140
return null;
139141
}
140142

@@ -263,7 +265,9 @@ private Profile getRecipientProfile(RecipientId recipientId, boolean force) {
263265
try {
264266
blockingGetProfile(retrieveProfile(recipientId, SignalServiceProfile.RequestType.PROFILE, false));
265267
} catch (IOException e) {
266-
logger.warn("Failed to retrieve profile, ignoring: {}", e.getMessage());
268+
logger.warn("Failed to retrieve profile for {}, ignoring: {}",
269+
context.getRecipientHelper().resolveSignalServiceAddress(recipientId).getIdentifier(),
270+
e.getMessage());
267271
}
268272

269273
return account.getProfileStore().getProfile(recipientId);
@@ -381,7 +385,9 @@ private Single<ProfileAndCredential> retrieveProfile(
381385

382386
logger.trace("Done handling retrieved profile");
383387
}).doOnError(e -> {
384-
logger.warn("Failed to retrieve profile, ignoring: {}", e.getMessage());
388+
logger.warn("Failed to retrieve profile for {}, ignoring: {}",
389+
context.getRecipientHelper().resolveSignalServiceAddress(recipientId).getIdentifier(),
390+
e.getMessage());
385391
final var profile = account.getProfileStore().getProfile(recipientId);
386392
final var newProfile = (
387393
profile == null ? Profile.newBuilder() : Profile.newBuilder(profile)

0 commit comments

Comments
 (0)