Uploaded image for project: 'Node.js Driver'
  1. Node.js Driver
  2. NODE-4816 Add error handling to the logger
  3. NODE-4848

Add runtime error handling

XMLWordPrintableJSON

    • Type: Icon: Sub-task Sub-task
    • Resolution: Fixed
    • Priority: Icon: Unknown Unknown
    • 6.4.0
    • Affects Version/s: None
    • Component/s: None
    • Labels:
    • 0
    • Not Needed
    • Not Needed
    • Hide

      1. What would you like to communicate to the user about this feature?
      2. Would you like the user to see examples of the syntax and/or executable code and its output?
      3. Which versions of the driver/connector does this apply to?

      Show
      1. What would you like to communicate to the user about this feature? 2. Would you like the user to see examples of the syntax and/or executable code and its output? 3. Which versions of the driver/connector does this apply to?

      Description

      Add error handling to runtime events, to ensure the logger never crashes the user's application.

      Acceptance Criteria

      • add try/catch block around write function
      • add try/catch block around EJSON stringify + default message when failure 
      • support
      • ensure all variables that the logger tries to access are defined properties
        • stricter typing for LoggableEvents
        • stricter typing within DefaultLogTransform (right now we only check typing through logObject.name property
      • Unit testing for all cases listed here

            Assignee:
            aditi.khare@mongodb.com Aditi Khare
            Reporter:
            andy.mina@mongodb.com Andy Mina
            Neal Beeken
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

              Created:
              Updated:
              Resolved: