Skip to content

Populate extra exception properties #8

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

Closed
niemyjski opened this issue May 20, 2015 · 1 comment
Closed

Populate extra exception properties #8

niemyjski opened this issue May 20, 2015 · 1 comment
Assignees

Comments

@niemyjski
Copy link
Member

We should be looking at the exception object and storing any extra properties for example for the error object looks like this:

{
    "stack": "xyz",
    "message": "test",
    "extraproperty": "test"
}

We should be storing extraproperty as an object in extended data as shown here: https://github.com/exceptionless/Exceptionless.Net/search?utf8=✓&q=extra We might need to create an exception that has this in c# and look at the json that is created so we ensure the added dictionary is correct for deserialization..

NOTE: There are known exception properties that should be ignored: https://github.com/csnover/TraceKit/blob/master/spec/fixtures/captured-errors.js

@niemyjski
Copy link
Member Author

@frankebersoll I've updated the description.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

2 participants