Skip to content

Commit

Permalink
Fix: Remove misleading error logs (#1222)
Browse files Browse the repository at this point in the history
Fixes #1221
  • Loading branch information
maciejwalkowiak authored Feb 4, 2021
1 parent 9f550e2 commit 1431c5e
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@
* Improve Kotlin compatibility for SdkVersion (#1213)
* Feat: Support logging via JUL (#1211)
* Fix: Fix returning Sentry trace header from Span (#1217)
* Fix: Remove misleading error logs (#1222)

# 4.0.0

Expand Down
3 changes: 0 additions & 3 deletions sentry/src/main/java/io/sentry/Hub.java
Original file line number Diff line number Diff line change
Expand Up @@ -312,7 +312,6 @@ public void setUser(final @Nullable User user) {
.log(SentryLevel.WARNING, "Instance is disabled and this 'setUser' call is a no-op.");
} else {
stack.peek().getScope().setUser(user);
options.getLogger().log(SentryLevel.FATAL, "Stack peek was null when setUser");
}
}

Expand All @@ -328,7 +327,6 @@ public void setFingerprint(final @NotNull List<String> fingerprint) {
options.getLogger().log(SentryLevel.WARNING, "setFingerprint called with null parameter.");
} else {
stack.peek().getScope().setFingerprint(fingerprint);
options.getLogger().log(SentryLevel.FATAL, "Stack peek was null when setFingerprint");
}
}

Expand All @@ -342,7 +340,6 @@ public void clearBreadcrumbs() {
"Instance is disabled and this 'clearBreadcrumbs' call is a no-op.");
} else {
stack.peek().getScope().clearBreadcrumbs();
options.getLogger().log(SentryLevel.FATAL, "Stack peek was null when clearBreadcrumbs");
}
}

Expand Down

0 comments on commit 1431c5e

Please sign in to comment.