diff --git a/sentry-core/src/main/java/io/sentry/core/MainEventProcessor.java b/sentry-core/src/main/java/io/sentry/core/MainEventProcessor.java index 6764a7704..cc5c8b45d 100644 --- a/sentry-core/src/main/java/io/sentry/core/MainEventProcessor.java +++ b/sentry-core/src/main/java/io/sentry/core/MainEventProcessor.java @@ -29,7 +29,7 @@ public SentryEvent process(SentryEvent event) { Throwable throwable = event.getThrowable(); if (throwable != null) { - event.setException(sentryExceptionFactory.getSentryExceptions(throwable)); + event.setExceptions(sentryExceptionFactory.getSentryExceptions(throwable)); } return event; diff --git a/sentry-core/src/main/java/io/sentry/core/Scope.java b/sentry-core/src/main/java/io/sentry/core/Scope.java index 1aacbcfee..d7c969bce 100644 --- a/sentry-core/src/main/java/io/sentry/core/Scope.java +++ b/sentry-core/src/main/java/io/sentry/core/Scope.java @@ -75,7 +75,7 @@ public void setTag(String key, String value) { this.tags.put(key, value); } - public Map getExtra() { + public Map getExtras() { return extra; } diff --git a/sentry-core/src/main/java/io/sentry/core/SentryClient.java b/sentry-core/src/main/java/io/sentry/core/SentryClient.java index c52f0eba8..96748586b 100644 --- a/sentry-core/src/main/java/io/sentry/core/SentryClient.java +++ b/sentry-core/src/main/java/io/sentry/core/SentryClient.java @@ -45,8 +45,8 @@ public SentryId captureEvent(SentryEvent event, @Nullable Scope scope) { if (event.getUser() == null) { event.setUser(scope.getUser()); } - if (event.getFingerprint() == null) { - event.setFingerprint(scope.getFingerprint()); + if (event.getFingerprints() == null) { + event.setFingerprints(scope.getFingerprint()); } if (event.getBreadcrumbs() == null) { event.setBreadcrumbs(new ArrayList<>(scope.getBreadcrumbs())); @@ -62,12 +62,12 @@ public SentryId captureEvent(SentryEvent event, @Nullable Scope scope) { } } } - if (event.getExtra() == null) { - event.setExtra(new HashMap<>(scope.getExtra())); + if (event.getExtras() == null) { + event.setExtras(new HashMap<>(scope.getExtras())); } else { - for (Map.Entry item : scope.getExtra().entrySet()) { - if (!event.getExtra().containsKey(item.getKey())) { - event.getExtra().put(item.getKey(), item.getValue()); + for (Map.Entry item : scope.getExtras().entrySet()) { + if (!event.getExtras().containsKey(item.getKey())) { + event.getExtras().put(item.getKey(), item.getValue()); } } } diff --git a/sentry-core/src/main/java/io/sentry/core/SentryEvent.java b/sentry-core/src/main/java/io/sentry/core/SentryEvent.java index 9d667eb59..2481702b7 100644 --- a/sentry-core/src/main/java/io/sentry/core/SentryEvent.java +++ b/sentry-core/src/main/java/io/sentry/core/SentryEvent.java @@ -32,6 +32,7 @@ public class SentryEvent implements IUnknownPropertiesConsumer { private Map tags; private Map extra; private Map unknown; + private Map modules; SentryEvent(SentryId eventId, Date timestamp) { this.eventId = eventId; @@ -119,11 +120,11 @@ public void setThreads(List threads) { this.threads = new SentryValues<>(threads); } - public List getException() { + public List getExceptions() { return exception.getValues(); } - public void setException(List exception) { + public void setExceptions(List exception) { this.exception = new SentryValues<>(exception); } @@ -187,11 +188,11 @@ public void setSdk(SdkVersion sdk) { this.sdk = sdk; } - public List getFingerprint() { + public List getFingerprints() { return fingerprint; } - public void setFingerprint(List fingerprint) { + public void setFingerprints(List fingerprint) { this.fingerprint = fingerprint; } @@ -225,11 +226,11 @@ public void setTag(String key, String value) { tags.put(key, value); } - public Map getExtra() { + public Map getExtras() { return extra; } - public void setExtra(Map extra) { + public void setExtras(Map extra) { this.extra = extra; } @@ -257,4 +258,12 @@ public void acceptUnknownProperties(Map unknown) { public Map getUnknown() { return unknown; } + + public Map getModules() { + return modules; + } + + public void setModules(Map modules) { + this.modules = modules; + } } diff --git a/sentry-core/src/main/java/io/sentry/core/protocol/Request.java b/sentry-core/src/main/java/io/sentry/core/protocol/Request.java index c736a0f4b..98dc51d62 100644 --- a/sentry-core/src/main/java/io/sentry/core/protocol/Request.java +++ b/sentry-core/src/main/java/io/sentry/core/protocol/Request.java @@ -62,19 +62,19 @@ public void setHeaders(Map headers) { this.headers = headers; } - public Map getEnv() { + public Map getEnvs() { return env; } - public void setEnv(Map env) { + public void setEnvs(Map env) { this.env = env; } - public Map getOther() { + public Map getOthers() { return other; } - public void setOther(Map other) { + public void setOthers(Map other) { this.other = other; } diff --git a/sentry-core/src/main/java/io/sentry/core/protocol/SdkVersion.java b/sentry-core/src/main/java/io/sentry/core/protocol/SdkVersion.java index 560a4fc11..5b9b2d07a 100644 --- a/sentry-core/src/main/java/io/sentry/core/protocol/SdkVersion.java +++ b/sentry-core/src/main/java/io/sentry/core/protocol/SdkVersion.java @@ -3,12 +3,14 @@ import io.sentry.core.IUnknownPropertiesConsumer; import java.util.List; import java.util.Map; +import java.util.concurrent.ConcurrentHashMap; import java.util.concurrent.CopyOnWriteArrayList; public class SdkVersion implements IUnknownPropertiesConsumer { private String name; private String version; private List packages = new CopyOnWriteArrayList<>(); + private Map integrations = new ConcurrentHashMap<>(); private Map unknown; public String getVersion() { @@ -45,4 +47,12 @@ public void addPackage(String name, String version) { public void acceptUnknownProperties(Map unknown) { this.unknown = unknown; } + + public Map getIntegrations() { + return integrations; + } + + public void setIntegrations(Map integrations) { + this.integrations = integrations; + } } diff --git a/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackFrame.java b/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackFrame.java index dc48c5cc7..3cbf642c6 100644 --- a/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackFrame.java +++ b/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackFrame.java @@ -25,6 +25,7 @@ public class SentryStackFrame implements IUnknownPropertiesConsumer { private Long symbolAddr; private Long instructionAddr; private Map unknown; + private String rawFunction; public List getPreContext() { return preContext; @@ -174,4 +175,12 @@ public void setNative(Boolean _native) { public void acceptUnknownProperties(Map unknown) { this.unknown = unknown; } + + public String getRawFunction() { + return rawFunction; + } + + public void setRawFunction(String rawFunction) { + this.rawFunction = rawFunction; + } } diff --git a/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackTrace.java b/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackTrace.java index 88f6b4b02..fe748c027 100644 --- a/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackTrace.java +++ b/sentry-core/src/main/java/io/sentry/core/protocol/SentryStackTrace.java @@ -7,7 +7,7 @@ /** The Sentry stacktrace. */ public class SentryStackTrace implements IUnknownPropertiesConsumer { private List frames; - // TODO registers is missing? + private Map registers; private Map unknown; public SentryStackTrace() {} @@ -37,4 +37,12 @@ public void setFrames(List frames) { public void acceptUnknownProperties(Map unknown) { this.unknown = unknown; } + + public Map getRegisters() { + return registers; + } + + public void setRegisters(Map registers) { + this.registers = registers; + } } diff --git a/sentry-core/src/main/java/io/sentry/core/protocol/User.java b/sentry-core/src/main/java/io/sentry/core/protocol/User.java index d88b363a7..fabe87a64 100644 --- a/sentry-core/src/main/java/io/sentry/core/protocol/User.java +++ b/sentry-core/src/main/java/io/sentry/core/protocol/User.java @@ -89,7 +89,7 @@ public void setIpAddress(String ipAddress) { * * @return the other user data. */ - public Map getOther() { + public Map getOthers() { return other; } @@ -98,7 +98,7 @@ public Map getOther() { * * @param other the other user related data.. */ - public void setOther(Map other) { + public void setOthers(Map other) { this.other = other; } diff --git a/sentry-core/src/test/java/io/sentry/core/HubTest.kt b/sentry-core/src/test/java/io/sentry/core/HubTest.kt index 29ca7963e..5e835622b 100644 --- a/sentry-core/src/test/java/io/sentry/core/HubTest.kt +++ b/sentry-core/src/test/java/io/sentry/core/HubTest.kt @@ -14,7 +14,7 @@ class HubTest { @Test fun `when cloning Scope it returns the same values`() { val scope = Scope(10) - scope.extra["extra"] = "extra" + scope.setExtra("extra", "extra") val breadcrumb = Breadcrumb() breadcrumb.message = "message" scope.addBreadcrumb(breadcrumb) @@ -29,7 +29,7 @@ class HubTest { val clone = scope.clone() assertNotNull(clone) assertNotSame(scope, clone) - assertEquals("extra", clone.extra["extra"]) + assertEquals("extra", clone.extras["extra"]) assertEquals("message", clone.breadcrumbs.first().message) assertEquals("transaction", scope.transaction) assertEquals("fingerprint", scope.fingerprint[0]) diff --git a/sentry-core/src/test/java/io/sentry/core/SentryClientTest.kt b/sentry-core/src/test/java/io/sentry/core/SentryClientTest.kt index 78d346199..5b1a4c67f 100644 --- a/sentry-core/src/test/java/io/sentry/core/SentryClientTest.kt +++ b/sentry-core/src/test/java/io/sentry/core/SentryClientTest.kt @@ -167,9 +167,9 @@ class SentryClientTest { sut.captureEvent(event, scope) assertEquals("message", event.breadcrumbs[0].message) - assertEquals("extra", event.extra["extra"]) + assertEquals("extra", event.extras["extra"]) assertEquals("tags", event.tags["tags"]) - assertEquals("fp", event.fingerprint[0]) + assertEquals("fp", event.fingerprints[0]) assertEquals("transaction", event.transaction) assertEquals("id", event.user.id) assertEquals(SentryLevel.FATAL, event.level) @@ -190,16 +190,16 @@ class SentryClientTest { assertEquals("message", event.breadcrumbs[1].message) // extras are appending - assertEquals("eventExtra", event.extra["eventExtra"]) - assertEquals("extra", event.extra["extra"]) + assertEquals("eventExtra", event.extras["eventExtra"]) + assertEquals("extra", event.extras["extra"]) // tags are appending assertEquals("eventTag", event.tags["eventTag"]) assertEquals("tags", event.tags["tags"]) // fingerprint is replaced - assertEquals("eventFp", event.fingerprint[0]) - assertEquals(1, event.fingerprint.size) + assertEquals("eventFp", event.fingerprints[0]) + assertEquals(1, event.fingerprints.size) assertEquals("eventTransaction", event.transaction) @@ -221,7 +221,7 @@ class SentryClientTest { sut.captureEvent(event, scope) // extras are appending - assertEquals("eventExtra", event.extra["eventExtra"]) + assertEquals("eventExtra", event.extras["eventExtra"]) // tags are appending assertEquals("eventTag", event.tags["eventTag"]) @@ -262,7 +262,7 @@ class SentryClientTest { }) setExtra("eventExtra", "eventExtra") setTag("eventTag", "eventTag") - fingerprint = listOf("eventFp") + fingerprints = listOf("eventFp") transaction = "eventTransaction" level = SentryLevel.DEBUG user = User().apply {