Uploaded image for project: 'Couchbase Go SDK'
  1. Couchbase Go SDK
  2. GOCBC-804

Long running http requests do not timeout correctly

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • None
    • 2.0.2
    • None
    • None
    • 1

    Description

      Http requests do not respect the timeout value specified. After changing how we handle context.Cancel on the request we no longer directly set the timeout and our timeout handler only ever runs after the http request has returned.

      Attachments

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

        Activity

          Ascending order - Click to sort in descending order
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-7.0.0-1464 contains gocbcore commit fb196d2 with commit message:
          GOCBC-804: Fix http request timeouts

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-1464 contains gocbcore commit fb196d2 with commit message: GOCBC-804 : Fix http request timeouts
          build-team Couchbase Build Team added a comment -

          Build couchbase-server-6.6.0-7624 contains gocbcore commit fb196d2 with commit message:
          GOCBC-804: Fix http request timeouts

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.0-7624 contains gocbcore commit fb196d2 with commit message: GOCBC-804 : Fix http request timeouts
          build-team Couchbase Build Team added a comment -

          Build sync_gateway-3.0.0-52 contains gocbcore commit fb196d2 with commit message:
          GOCBC-804: Fix http request timeouts

          build-team Couchbase Build Team added a comment - Build sync_gateway-3.0.0-52 contains gocbcore commit fb196d2 with commit message: GOCBC-804 : Fix http request timeouts

          People

            charles.dixon Charles Dixon
            charles.dixon Charles Dixon
            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