mirror of
https://github.com/ethereum/go-ethereum.git
synced 2026-06-12 09:51:36 +00:00
cmd/pushtx: address code review feedback
Co-authored-by: drQedwards <213266729+drQedwards@users.noreply.github.com>
This commit is contained in:
parent
17ef318c42
commit
b91a437b09
2 changed files with 14 additions and 4 deletions
|
|
@ -85,7 +85,7 @@ func run(args []string, stdin io.Reader) error {
|
|||
txHex = strings.TrimSpace(string(data))
|
||||
}
|
||||
if txHex == "" {
|
||||
return fmt.Errorf("no transaction data provided\nUsage: pushtx [--rpc URL] <tx-hex>")
|
||||
return fmt.Errorf("no transaction data provided (see --help for usage)")
|
||||
}
|
||||
|
||||
rawTx, err := hex.DecodeString(strings.TrimPrefix(txHex, "0x"))
|
||||
|
|
|
|||
|
|
@ -98,10 +98,20 @@ func fakeRPC(t *testing.T, wantErr bool) *httptest.Server {
|
|||
}
|
||||
// Decode the raw tx to return its hash as the result.
|
||||
var hexData string
|
||||
json.Unmarshal(req.Params[0], &hexData)
|
||||
rawBytes, _ := hexutil.Decode(hexData)
|
||||
if err := json.Unmarshal(req.Params[0], &hexData); err != nil {
|
||||
http.Error(w, err.Error(), http.StatusBadRequest)
|
||||
return
|
||||
}
|
||||
rawBytes, err := hexutil.Decode(hexData)
|
||||
if err != nil {
|
||||
http.Error(w, err.Error(), http.StatusBadRequest)
|
||||
return
|
||||
}
|
||||
var tx types.Transaction
|
||||
tx.UnmarshalBinary(rawBytes)
|
||||
if err := tx.UnmarshalBinary(rawBytes); err != nil {
|
||||
http.Error(w, err.Error(), http.StatusBadRequest)
|
||||
return
|
||||
}
|
||||
json.NewEncoder(w).Encode(map[string]interface{}{
|
||||
"jsonrpc": "2.0",
|
||||
"id": req.ID,
|
||||
|
|
|
|||
Loading…
Reference in a new issue