Skip to content

Commit e7dc682

Browse files
authored
Merge 1035e1c into 51e7326
2 parents 51e7326 + 1035e1c commit e7dc682

2 files changed

Lines changed: 5 additions & 3 deletions

File tree

CONTRIBUTING.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ We suggest opening an issue to discuss bigger changes before investing on a big
55

66
# Requirements
77

8-
The project currently requires you run JDK 11.
8+
The project currently requires you run JDK 17.
99

1010
## Android
1111

sentry-android-core/src/main/java/io/sentry/android/core/DefaultAndroidEventProcessor.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ public DefaultAndroidEventProcessor(
9494

9595
ExecutorService executorService = Executors.newSingleThreadExecutor();
9696
// dont ref. to method reference, theres a bug on it
97-
contextData = executorService.submit(() -> loadContextData());
97+
contextData = executorService.submit(this::loadContextData);
9898

9999
executorService.shutdown();
100100
}
@@ -282,7 +282,9 @@ private void setArchitectures(final @NotNull Device device) {
282282
// TODO: missing usable memory
283283

284284
Device device = new Device();
285-
device.setName(getDeviceName());
285+
if (!options.isSendDefaultPii()) {
286+
device.setName(getDeviceName());
287+
}
286288
device.setManufacturer(Build.MANUFACTURER);
287289
device.setBrand(Build.BRAND);
288290
device.setFamily(getFamily());

0 commit comments

Comments
 (0)