Skip to content
This repository was archived by the owner on Oct 20, 2024. It is now read-only.

Commit 59b62ef

Browse files
committed
Return both error and success responses from a broadcast
1 parent 8814607 commit 59b62ef

File tree

1 file changed

+19
-9
lines changed

1 file changed

+19
-9
lines changed

flashbotsrpc.go

+19-9
Original file line numberDiff line numberDiff line change
@@ -777,10 +777,11 @@ func (broadcaster *BuilderBroadcastRPC) BroadcastBundle(privKey *ecdsa.PrivateKe
777777
for _, requestResponse := range requestResponses {
778778
if requestResponse.Err != nil {
779779
responses = append(responses, BuilderBroadcastResponse{Err: requestResponse.Err})
780+
} else {
781+
fbResponse := FlashbotsSendBundleResponse{}
782+
err := json.Unmarshal(requestResponse.Msg, &fbResponse)
783+
responses = append(responses, BuilderBroadcastResponse{BundleResponse: fbResponse, Err: err})
780784
}
781-
fbResponse := FlashbotsSendBundleResponse{}
782-
err := json.Unmarshal(requestResponse.Msg, &fbResponse)
783-
responses = append(responses, BuilderBroadcastResponse{BundleResponse: fbResponse, Err: err})
784785
}
785786

786787
return responses
@@ -872,19 +873,28 @@ func (broadcaster *BuilderBroadcastRPC) broadcastRequest(method string, privKey
872873
errorResp := new(RelayErrorResponse)
873874
if err := json.Unmarshal(data, errorResp); err == nil && errorResp.Error != "" {
874875
// relay returned an error
875-
responseArr := []broadcastRequestResponse{{Msg: nil, Err: fmt.Errorf("%w: %s", ErrRelayErrorResponse, errorResp.Error)}}
876-
return responseArr
876+
responses = append(responses, broadcastRequestResponse{
877+
Msg: nil,
878+
Err: fmt.Errorf("%w: %s", ErrRelayErrorResponse, errorResp.Error),
879+
})
880+
continue
877881
}
878882

879883
resp := new(rpcResponse)
880884
if err := json.Unmarshal(data, resp); err != nil {
881-
responseArr := []broadcastRequestResponse{{Msg: nil, Err: err}}
882-
return responseArr
885+
responses = append(responses, broadcastRequestResponse{
886+
Msg: nil,
887+
Err: err,
888+
})
889+
continue
883890
}
884891

885892
if resp.Error != nil {
886-
responseArr := []broadcastRequestResponse{{Msg: nil, Err: fmt.Errorf("%w: %s", ErrRelayErrorResponse, (*resp).Error.Message)}}
887-
return responseArr
893+
responses = append(responses, broadcastRequestResponse{
894+
Msg: nil,
895+
Err: fmt.Errorf("%w: %s", ErrRelayErrorResponse, (*resp).Error.Message),
896+
})
897+
continue
888898
}
889899

890900
responses = append(responses, broadcastRequestResponse{

0 commit comments

Comments
 (0)