Opened 6 years ago

Closed 6 years ago

#3314 closed defect (fixed)

Add a catch-all to logger Formatter destructor

Reported by: muks Owned by: muks
Priority: medium Milestone: bind10-1.2-release-freeze
Component: logging Version:
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: N/A
Sub-Project: DNS Feature Depending on Ticket:
Estimated Difficulty: 0 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

This is an offshoot of #1966. In the short term, we add a catch-all block to liblog's ~Formatter() so that exceptions are not propagated from the destructor. See #1966 for the commentary.

Subtickets

Change History (3)

comment:1 Changed 6 years ago by muks

  • Owner set to UnAssigned
  • Status changed from new to reviewing

Up for review.

comment:2 Changed 6 years ago by kean

  • Owner changed from UnAssigned to muks

This looks fine, and does exactly as discussed in #1966. Please merge and close.

comment:3 Changed 6 years ago by muks

  • Resolution set to fixed
  • Status changed from reviewing to closed

Merged to master branch in commit 3327e16875dc619a549247cb4dee855be5d538d5:

* 7058dd1 [3314] Add a catch-all to logger Formatter destructor

Resolving as fixed. Thank you for the review Kean.

Note: See TracTickets for help on using tickets.