Skip to content

fix(util/gconv): support array-backed unmarshalling values - #4810

Open
ErzerLP wants to merge 1 commit into
gogf:masterfrom
ErzerLP:fix/gconv-uuid-slice
Open

fix(util/gconv): support array-backed unmarshalling values#4810
ErzerLP wants to merge 1 commit into
gogf:masterfrom
ErzerLP:fix/gconv-uuid-slice

Conversation

@ErzerLP

@ErzerLP ErzerLP commented Jul 26, 2026

Copy link
Copy Markdown

Summary

  • handle addressable array-backed values through the existing common-interface unmarshalling path
  • add a focused regression test for string slices bound to array-backed UnmarshalText values
  • record the mandatory red-green bug-fix gate and the verified conversion behavior in OpenSpec

Test plan

  • Before the production fix, go test -count=1 -run '^Test_Issue4786$' ./util/gconv failed with reflect: call of reflect.Value.IsNil on array Value
  • After the fix, the same targeted command passed
  • go test -count=1 -race ./util/gconv/...
  • direct conversion into []uuid.UUID
  • make lint
  • openspec validate --all --strict

Fixes #4786

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.

net/http: 无法将 UUID slice 作为参数传入到 HTTP 服务

1 participant