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

Long running http requests do not timeout correctly

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2.0.2
    • Component/s: None
    • Labels:
      None
    • Story Points:
      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

          Hide
          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

          Show
          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
          Hide
          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

          Show
          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
          Hide
          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

          Show
          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

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