knyk-dev opened a new pull request, #1070: URL: https://github.com/apache/incubator-seata-go/pull/1070
<!-- Thanks for sending a pull request! --> <!-- Please make sure you have read and understood the contributing guidelines --> - [ ] I have registered the PR [changes](https://github.com/apache/incubator-seata-go/tree/master/changes). **What this PR does**: **Which issue(s) this PR fixes**: Fixes #1069 **Special notes for your reviewer**: The bug is in the `valueLength == 0` branch of `decodeHeapMap()`. It clears `key` instead of keeping the decoded key and setting `value` to an empty string. And the change is intentionally small and only touches the Getty read/write path plus a regression test. **Does this PR introduce a user-facing change?**: NONE <!-- If no, just write "NONE" in the release-note block below. If yes, a release note is required: Enter your extended release note in the block below. If the PR requires additional action from users switching to the new release, include the string "action required". --> ```release-note fix: preserve HeadMap keys when decoding empty header values in Getty RPC messages ``` -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
