Skip to content

Msgpack fix 😡#40

Merged
btoews merged 2 commits intomainfrom
msgpack-fix
Mar 19, 2025
Merged

Msgpack fix 😡#40
btoews merged 2 commits intomainfrom
msgpack-fix

Conversation

@btoews
Copy link
Contributor

@btoews btoews commented Mar 19, 2025

This is apparently expected behavior?

func TestMsgpack(t *testing.T) {
	v := map[uint64]uint64{1: 1}
	b, err := msgpack.Marshal(v)
	assert.NoError(t, err)

	var vv any
	assert.NoError(t, msgpack.Unmarshal(b, &vv))
}
--- FAIL: TestMsgpack (0.00s)
    main_test.go:16: Did not expect an error but got:
        msgpack: invalid code=cf decoding string/bytes length

@btoews btoews merged commit 8578343 into main Mar 19, 2025
1 check passed
@btoews btoews deleted the msgpack-fix branch March 19, 2025 18:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant