Skip to content

Commit

Permalink
fix: builder args incorrectly configured (#343)
Browse files Browse the repository at this point in the history
as new args were added to geth the hardcoded indexes are not configured
properly. This makes cmd args for builder more robust.
  • Loading branch information
avalonche authored Oct 27, 2023
1 parent 789ed8e commit 66e73fb
Showing 1 changed file with 5 additions and 2 deletions.
7 changes: 5 additions & 2 deletions src/el/geth/geth_launcher.star
Original file line number Diff line number Diff line change
Expand Up @@ -223,8 +223,11 @@ def get_config(
]

if BUILDER_IMAGE_STR in image:
cmd[10] = "--http.api=admin,engine,net,eth,web3,debug,flashbots"
cmd[14] = "--ws.api=admin,engine,net,eth,web3,debug,flashbots"
for index, arg in enumerate(cmd):
if "--http.api" in arg:
cmd[index] = "--http.api=admin,engine,net,eth,web3,debug,mev,flashbots"
if "--ws.api" in arg:
cmd[index] = "--ws.api=admin,engine,net,eth,web3,debug,mev,flashbots"

if len(existing_el_clients) > 0:
cmd.append(
Expand Down

0 comments on commit 66e73fb

Please sign in to comment.