tree de2158b0c56207574ac5415862a7c18e0a4bfa75
parent 0f592d2728bb767204b019bf926fbcd791267115
author Ian Cottrell <iancottrell@google.com> 1592601729 -0400
committer Ian Cottrell <iancottrell@google.com> 1592838058 +0000

internal/jsonrpc2: don't set result in response if there is an error

Marshalling an empty raw message produces a null rather than omitting the field
Instead we do not set the field unless there is no error.
Includes a test that reproduced the issue before the fix.

Fixes golang/go#39719

Change-Id: I683b8ea55d3425613fc956993280ff51202f49fc
Reviewed-on: https://go-review.googlesource.com/c/tools/+/239097
Run-TryBot: Ian Cottrell <iancottrell@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Robert Findley <rfindley@google.com>