Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

v0.3.2 latest op-node emits [: Illegal number: with error #122

Closed
HiroyukiNaito opened this issue Oct 6, 2023 · 2 comments · Fixed by #123
Closed

v0.3.2 latest op-node emits [: Illegal number: with error #122

HiroyukiNaito opened this issue Oct 6, 2023 · 2 comments · Fixed by #123
Labels
minor This is a minor bug state: investigation This is being reviewed type: bug Something isn't working

Comments

@HiroyukiNaito
Copy link

v0.3.2 latest op-node emits [: Illegal number: with error and down the container

node-node-1  | ./op-node-entrypoint: 5: Bad substitution
node-node-1  | ./op-node-entrypoint: 5: [: Illegal number:
@HiroyukiNaito
Copy link
Author

HiroyukiNaito commented Oct 6, 2023

It looks like op-node-entrypoint file has a bug.

bash substitution don't work in following line.

https://github.com/base-org/node/blob/main/op-node-entrypoint#L5

In the mean time, I changed the following code and worked.

#!/bin/bash
set -eu

# wait until local geth comes up (authed so will return 401 without token)
OP_NODE_L2_ENGINE_RPC_HTTP=$(echo ${OP_NODE_L2_ENGINE_RPC} | sed -e 's/ws/http/g')
until [ "$(curl -s -w '%{http_code}' -o /dev/null "${OP_NODE_L2_ENGINE_RPC_HTTP}")" -eq 401 ]; do
  echo "waiting for geth to be ready"
  sleep 5
done

# public-facing P2P node, advertise public IP address
PUBLIC_IP=$(curl -s v4.ident.me)
export OP_NODE_P2P_ADVERTISE_IP=$PUBLIC_IP

echo "$OP_NODE_L2_ENGINE_AUTH_RAW" > "$OP_NODE_L2_ENGINE_AUTH"

exec ./op-node --rpc.enable-admin

@wbnns wbnns added minor This is a minor bug type: bug Something isn't working state: investigation This is being reviewed labels Oct 6, 2023
@wbnns
Copy link
Member

wbnns commented Oct 6, 2023

@HiroyukiNaito

Thanks for mentioning and also sharing the workaround; we'll check into this and reply here as soon as we have an update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
minor This is a minor bug state: investigation This is being reviewed type: bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants