Uploaded image for project: 'Couchbase Gateway'
  1. Couchbase Gateway
  2. CBG-309

Don't log expected REST errors to sg_error.log

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.5.0
    • 2.6.0
    • SyncGateway
    • Security Level: Public
    • None
    • CBG Sprint 20
    • 3

    Description

      The changes in https://github.com/couchbase/sync_gateway/pull/3593 incorrectly added ERR level logging for expected REST API errors (key not found on GET, etc).

      Error logging should be reserved for critical errors that require user attention.  

      In handler.go, h.writeError should be modified to only log the error if status is greater than or equal to 500.

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          Build sync_gateway-2.6.0-23 contains sync_gateway commit 19963cf with commit message:
          CBG-309 - Only log errors returned from HTTP handlers for status >= 500 (#4086)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-23 contains sync_gateway commit 19963cf with commit message: CBG-309 - Only log errors returned from HTTP handlers for status >= 500 (#4086)

          Build sync_gateway-2.6.0-23 contains sync_gateway commit 19963cf with commit message:
          CBG-309 - Only log errors returned from HTTP handlers for status >= 500 (#4086)

          build-team Couchbase Build Team added a comment - Build sync_gateway-2.6.0-23 contains sync_gateway commit 19963cf with commit message: CBG-309 - Only log errors returned from HTTP handlers for status >= 500 (#4086)

          People

            ben.brooks Ben Brooks
            adamf Adam Fraser
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty