Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-22796

unable to unpack dcp response with xattrs: unpack requires a string argument of length 4

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Major
    • 5.0.0
    • 5.0.0
    • test-execution
    • None
    • Untriaged
    • Ubuntu 64-bit
    • Unknown

    Description

      Hi David Haikney
      to test dcp with xattrs I used your PR as base line https://github.com/couchbaselabs/pydcp/pull/25/files

      I've slightly updated it and added some new tests/functionality.

      so, when I run my tests (https://github.com/andreibaranouski/pydcp/commits/xattr_dcp) I get errors like:

      unpack requires a string argument of length 4
      unpack requires a string argument of length 4
      unpack requires a string argument of length 4
      unpack requires a string argument of length 4
      unpack requires a string argument of length 4
      8 0
      unpack requires a string argument of length 4
      stream.last_by_seqno: unpack requires a string argument of length 419
      stream.last_by_seqno:
       unpack requires a string argument of length 4unpack requires a string argument of length 4
       
      unpack requires a string argument of length 4stream.last_by_seqno:19
      

      when
      if (dtype & DATATYPE_XATTR):
      + total_xattr_len = struct.unpack('>I',value[0:4])[0] - FAILED

      could you please to take a look on my tests and check what is wrong there: dcp_client, test or the issue on CB side

      how to run the tests:
      1) get https://github.com/andreibaranouski/pydcp/commits/xattr_dcp
      2) python pydcp -h 127.0.0.1:8091 -b cb -x -u root -p couchbase -s XATTRTestCases

      Attachments

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

        Activity

          People

            andreibaranouski Andrei Baranouski
            andreibaranouski Andrei Baranouski
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty