Skip to content

Commit

Permalink
fix(member.unpack): fix payload position (#62)
Browse files Browse the repository at this point in the history
Commit 7021e9f refactored
members.unpack function but introduced a typo with the position of the
payload in the array that forced payload to be ignored.

Such bug broke anti_entropy step, forcing incarnation to be updated
and payload to be ignored, making instances view of each other
inconsistent.

Closes #61
  • Loading branch information
VifleY authored Apr 9, 2024
1 parent cbdd0b3 commit a910a22
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,10 @@ and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.

## [Unreleased]

### Fixed

- Invalid payload parsing in anti entropy step.

## [2.4.3] - 2024-01-29

### Fixed
Expand Down
2 changes: 1 addition & 1 deletion membership/members.lua
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ end

function members.unpack(member)
checks('table')
local payload = member[3]
local payload = member[4]
if payload == msgpack.NULL
or type(payload) ~= 'table'
then
Expand Down

0 comments on commit a910a22

Please sign in to comment.