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

Enable autocollection of unhandled exceptions #963

Merged
merged 1 commit into from
May 24, 2021

Conversation

millicentachieng
Copy link
Contributor

@millicentachieng millicentachieng commented May 21, 2021

Overview

We've been missing some errors in our telemetry. Enabling autocollection of uncaught exceptions will help us see more errors that our users are encountering while interacting with the app.

I have made an attempt at sanitizing the stack trace and only storing what we need. The data collected for an unhandled exception looks like this:

{
    "baseType": "ExceptionData",
    "baseData": {
        "ver": 2,
        "exceptions": [
            {
                "typeName": "TypeError",
                "message": "UNHANDLED_ERROR",
                "hasFullStack": false,
                "stack": null,
                "parsedStack": [
                    {
                        "level": 0,
                        "method": "getStyleFor",
                        "assembly": "at getStyleFor (webpack-internal:///./src/app/utils/badge-color.ts:7:19)",
                        "fileName": "webpack-internal:///./src/app/utils/badge-color.ts",
                        "line": 7
                    }
                ]
            }
        ],
        "severityLevel": 3,
        "properties": {
            "ComponentName": "at getStyleFor (webpack-internal:///./src/app/utils/badge-color.ts:7:19)",
            "Message": "TypeError: Cannot read property 'toUpperCase' of null",
            "ApplicationName": "Graph Explorer v4",
            "IsAuthenticated": "true"
        }
    }
}

@github-actions
Copy link
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://jolly-sand-0ac78c710-963.centralus.azurestaticapps.net

@millicentachieng millicentachieng linked an issue May 24, 2021 that may be closed by this pull request
@millicentachieng millicentachieng merged commit 81b0bab into dev May 24, 2021
@millicentachieng millicentachieng deleted the feature/telemetry-enable-error-autocollection branch May 24, 2021 08:45
thewahome added a commit that referenced this pull request Jun 8, 2021
* Task: Add Cred Scan (#937)

* Task: telemetry capture English theme names only (#958)

* Feature: add telemetry middleware (#961)

* Task: Capture deployed version of application for all telemetry data collected (#964)

* Task: Enable autocollection of unhandled exceptions (#963)

* Fix: prevent running empty requests (#972)

* Task: add code-owners (#969)

* Task: send notification when release is created (#967)

* Fix: interaction in progress (#976)

* Feature: Display Graph Explorer Version (#977)

* Feature: show delta links when property is present (#978)

* Feature: Intergrate OCV feedback  (#923)

* Fix: mgt story links (#981)

* Task: Update README.md for Azure auth steps (#982)
thewahome added a commit that referenced this pull request Jun 10, 2021
* Task: Update version number (#951)

* Task: telemetry capture English theme names only (#958)

* Feature: add telemetry middleware (#961)

* Task: Capture deployed version of application for all telemetry data collected (#964)

* Task: Enable autocollection of unhandled exceptions (#963)

* Fix: prevent running empty requests (#972)

* Task: add code-owners (#969)

* Task: send notification when release is created (#967)

* Fix: interaction in progress (#976)

* Feature: Display Graph Explorer Version (#977) [Disabled]

* Feature: show delta links when property is present (#978)

* Fix: mgt story links (#981)

* Task: Update README.md for Azure auth steps (#982)
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.

Enable autocollection of unhandled exceptions
2 participants