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

DocumentChanged webhook _rev empty

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 2.7.0
    • 2.7.0
    • SyncGateway
    • Security Level: Public
    • None
    • CBG Sprint 32
    • 1

    Description

      I suspect this is down to the same root cause asĀ CBG-522, so perhaps doing that first would resolve this.

      The "_rev" property of a document_changed webhook is empty in 2.7.0. This works in 2.6.0.

      Steps to reproduce

      1. Start SG with a document_changed webhook set up in the config
      2. Create a document
      3. Update the document
      4. Observe that the webhook body sent in step 3 did not contain the rev id

      2.6.0:

      2.7.0:

      Attachments

        Issue Links

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

          Activity

            ben.brooks Ben Brooks added a comment -

            Inside MarshalBodyForWebhook, we're inserting doc.RevID, and actually this should be doc.CurrentRev

            ben.brooks Ben Brooks added a comment - Inside MarshalBodyForWebhook, we're inserting doc.RevID, and actually this should be doc.CurrentRev

            Build sync_gateway-2.7.0-50 contains sync_gateway commit c1f8583 with commit message:
            CBG-521: DocumentChanged fix _rev missing (#4253)

            build-team Couchbase Build Team added a comment - Build sync_gateway-2.7.0-50 contains sync_gateway commit c1f8583 with commit message: CBG-521 : DocumentChanged fix _rev missing (#4253)

            People

              jacques.rascagneres Jacques Rascagneres
              ben.brooks Ben Brooks
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty