[MB-44534] SetWithMeta allows docs with datatype:xattr and empty payload - Couchbase database

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Critical
Resolution: Fixed
Affects Version/s: 6.6.1
Fix Version/s: 6.6.2
Component/s: couchbase-bucket
Labels:
- approved-for-6.6.2
- releasenote

Triage:
Triaged
Story Points:
1
Is this a Regression?:
Unknown

Description

Proved in the unit test at http://review.couchbase.org/c/kv_engine/+/146827

Attachments

Gerrit Reviews

- Issue Only
- Show All Reviews
- Show Open Reviews
- Show All Issues
- Show Open Issues

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

Couchbase Build Team added a comment - 26/Feb/21 2:15 AM

Build couchbase-server-6.6.2-9547 contains kv_engine commit 21e5ea5 with commit message:
~~MB-44534~~: DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 26/Feb/21 2:15 AM Build couchbase-server-6.6.2-9547 contains kv_engine commit 21e5ea5 with commit message: MB-44534 : DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 26/Feb/21 2:15 AM

Build couchbase-server-6.6.2-9547 contains kv_engine commit 0f1701d with commit message:
~~MB-44534~~: SetWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 26/Feb/21 2:15 AM Build couchbase-server-6.6.2-9547 contains kv_engine commit 0f1701d with commit message: MB-44534 : SetWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 01/Mar/21 7:49 AM

Build couchbase-server-7.0.0-4570 contains kv_engine commit 0f1701d with commit message:
~~MB-44534~~: SetWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 01/Mar/21 7:49 AM Build couchbase-server-7.0.0-4570 contains kv_engine commit 0f1701d with commit message: MB-44534 : SetWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 02/Mar/21 3:02 AM

Build couchbase-server-7.0.0-4580 contains kv_engine commit 21e5ea5 with commit message:
~~MB-44534~~: DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 02/Mar/21 3:02 AM Build couchbase-server-7.0.0-4580 contains kv_engine commit 21e5ea5 with commit message: MB-44534 : DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 03/Mar/21 6:30 PM

Build couchbase-server-7.0.0-4603 contains kv_engine commit 21e5ea5 with commit message:
~~MB-44534~~: DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 03/Mar/21 6:30 PM Build couchbase-server-7.0.0-4603 contains kv_engine commit 21e5ea5 with commit message: MB-44534 : DelWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 03/Mar/21 6:30 PM

Build couchbase-server-7.0.0-4603 contains kv_engine commit 0f1701d with commit message:
~~MB-44534~~: SetWithMeta fixes the datatype for empty value with dt:xattr

Couchbase Build Team added a comment - 03/Mar/21 6:30 PM Build couchbase-server-7.0.0-4603 contains kv_engine commit 0f1701d with commit message: MB-44534 : SetWithMeta fixes the datatype for empty value with dt:xattr

Paolo Cocchi added a comment - 31/Mar/21 2:58 AM

Description for release notes
Summary: The SetWithMeta MCBP operation allowed to store empty documents with invalid datatype. Fixed by sanitizing the payload: the datatype is set to RAW as per KV invariant.

Paolo Cocchi added a comment - 31/Mar/21 2:58 AM Description for release notes Summary: The SetWithMeta MCBP operation allowed to store empty documents with invalid datatype. Fixed by sanitizing the payload: the datatype is set to RAW as per KV invariant.