Skip to content

Commit

Permalink
fix: remove eth_subscription hack
Browse files Browse the repository at this point in the history
  • Loading branch information
Stebalien committed Feb 17, 2023
1 parent 706e1cc commit fa8294b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion cli/util/api.go
Original file line number Diff line number Diff line change
Expand Up @@ -343,7 +343,7 @@ func GetFullNodeAPIV1(ctx *cli.Context, opts ...GetFullNodeOption) (v1api.FullNo

var rpcOpts []jsonrpc.Option
if options.ethSubHandler != nil {
rpcOpts = append(rpcOpts, jsonrpc.WithClientHandler("Filecoin", options.ethSubHandler), jsonrpc.WithClientHandlerAlias("eth_subscription", "Filecoin.EthSubscription"))
rpcOpts = append(rpcOpts, jsonrpc.WithClientHandler("Filecoin", options.ethSubHandler))
}

heads, err := GetRawAPIMulti(ctx, repo.FullNode, "v1")
Expand Down
1 change: 0 additions & 1 deletion itests/kit/rpc.go
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,6 @@ func fullRpc(t *testing.T, f *TestFullNode) (*TestFullNode, Closer) {

rpcOpts := []jsonrpc.Option{
jsonrpc.WithClientHandler("Filecoin", f.EthSubRouter),
jsonrpc.WithClientHandlerAlias("eth_subscription", "Filecoin.EthSubscription"),
}

cl, stop, err := client.NewFullNodeRPCV1(context.Background(), "ws://"+srv.Listener.Addr().String()+"/rpc/v1", nil, rpcOpts...)
Expand Down

0 comments on commit fa8294b

Please sign in to comment.