Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow the user to specify a message with exceptions #77

Merged
merged 2 commits into from
Sep 23, 2016

Conversation

marcomorain
Copy link
Collaborator

@marcomorain marcomorain commented Sep 16, 2016

It is often useful to add a string of context along with
an exception. This helps show the user what was happening
when the exception occurred.

Use case:

try {
  // network operation to load cats
} catch (HttpException e) {
  Sentry.captureException(e, "Error loading cats");
}

It is often useful to add a string of context along with
and exception. This helps show the user what was happening
when the exception occured.

Use case:

try {
  // network operation to load cats
} catch (HttpException e) {
  Sentry.captureException(e, "Error loading cats");
}
@@ -689,7 +691,7 @@ public boolean equals(Object other) {
// dalvik.system.*
static final String isInternalPackage = "^(java|android|com\\.android|com\\.google\\.android|dalvik\\.system)\\..*";

private final static SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss");
private final static SimpleDateFormat sdf = new SimpleDateFormat("yyyy-MM-dd'T'HH:mm:ss", Locale.US);
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

~ this fixes a warning in AndroidStudio

@marcomorain marcomorain merged commit a99f44e into joshdholtz:master Sep 23, 2016
@marcomorain marcomorain deleted the feature-allow-message branch September 23, 2016 21:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants