[COMPASS-5038] Log uncaught exceptions Created: 25/Aug/21  Updated: 29/Oct/23  Resolved: 14/Sep/21

Status: Closed
Project: Compass
Component/s: None
Affects Version/s: None
Fix Version/s: 1.29.4

Type: Task Priority: Major - P3
Reporter: Anna Henningsen Assignee: Anna Henningsen
Resolution: Fixed Votes: 0
Labels: None
Remaining Estimate: Not Specified
Time Spent: Not Specified
Original Estimate: Not Specified

Issue Links:
Depends
depends on COMPASS-5031 Create compass-logging helper package... Closed
Epic Link: COMPASS-4361
Story Points: 2
Documentation Changes: Not Needed
Sprint: Iteration Kampala

 Description   

Uncaught exceptions are always logged, using the existing handling for the compass:error:fatal event in the compass package. They are logged on the main MongoLogWriter instance using the .fatal() method.



 Comments   
Comment by Githook User [ 14/Dec/21 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: chore(cli-repl): add tls logging regression test COMPASS-5038
Branch: 5038-dev-2
https://github.com/mongodb-js/mongosh/commit/eb6de67f8592318075d3e254e4ea7c845d60f4c3

Comment by Githook User [ 14/Sep/21 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}

Message: chore(compass-e2e-tests): add logging tests COMPASS-5038 (#2453)

Also make a minor adjustment to the `compass:error:fatal`
handler to handle the case where `error` is not set on the
`ErrorEvent` (this happens e.g. when an asynchronous exception
comes from the built-in inspect because that is considered
a cross-context operation).
Branch: COMPASS-5046-add-e2e-fixture
https://github.com/mongodb-js/compass/commit/fc02481c92be568067c573f427ed7d704a002b18

Comment by Githook User [ 14/Sep/21 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna.henningsen@mongodb.com', 'username': 'addaleax'}

Message: chore(compass-e2e-tests): add logging tests COMPASS-5038 (#2453)

Also make a minor adjustment to the `compass:error:fatal`
handler to handle the case where `error` is not set on the
`ErrorEvent` (this happens e.g. when an asynchronous exception
comes from the built-in inspect because that is considered
a cross-context operation).
Branch: main
https://github.com/mongodb-js/compass/commit/fc02481c92be568067c573f427ed7d704a002b18

Comment by Githook User [ 13/Sep/21 ]

Author:

{'name': 'Anna Henningsen', 'email': 'anna@addaleax.net', 'username': 'addaleax'}

Message: chore(compass-e2e-tests): add logging tests COMPASS-5038

Also make a minor adjustment to the `compass:error:fatal`
handler to handle the case where `error` is not set on the
`ErrorEvent` (this happens e.g. when an asynchronous exception
comes from the built-in inspect because that is considered
a cross-context operation).
Branch: 5038-dev
https://github.com/mongodb-js/compass/commit/f9436e623a8826d77d279923fdff0b498654c54b

Generated at Wed Feb 07 22:38:14 UTC 2024 using Jira 9.7.1#970001-sha1:2222b88b221c4928ef0de3161136cc90c8356a66.