Missing Application Insights env var results in unexpected page render
rocketnova opened this issue · comments
Description
While working on #432, I discovered that when the App Insights environment variable is missing, CST displays the following:
This is unexpected.
Acceptance Criteria
- Identify what's going on
- Make any necessary fix so that a missing App Insights env var renders nicely
@lomky (CC @nicoleslaw @dianagriffin). It turns out that when an error is thrown and we do not catch it, this is what is displayed (see ticket description). Is this an issue we want to fix? I think it's pretty rare, but still possible that we may have uncaught errors and we probably want to display something nicer. If so, can you open a new ticket for it?
Details: It turns out that this throw new Error
wasn't getting caught and was resulting in this render.
ui-claim-tracker/utils/logger.ts
Lines 81 to 84 in 72d5437
I can open a ticket on this one. @dianagriffin do you think it makes sense to display the 500 message here even though it's not a 500 error?
Relates to #467
@nicoleslaw I agree that the 500 message is generic enough to make sense/be generally accurate from the user perspective.