Skip to content

Commit

Permalink
feat: strip trailing slash from beacon node URL in checkpoint sync
Browse files Browse the repository at this point in the history
fix(integration.yaml): set beacon_node_url to MAINNET_BEACON_API_URL
  • Loading branch information
samcm committed Feb 21, 2025
1 parent 5852cd5 commit a2585e9
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 2 deletions.
7 changes: 6 additions & 1 deletion .github/actions/checkpoint-sync/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -29,6 +29,11 @@ runs:
- name: Configure checkpointz
shell: bash
run: |
# Strip trailing slash from the beacon node URL
beacon_node_url="${{ inputs.beacon_node_url }}"
if [[ "$beacon_node_url" == *"/" ]]; then
beacon_node_url="${beacon_node_url%/}"
fi
cat <<EOF > checkpointz.yaml
global:
listenAddr: ":5555"
Expand All @@ -37,7 +42,7 @@ runs:
beacon:
upstreams:
- name: state-provider
address: ${{ inputs.beacon_node_url }}
address: "${beacon_node_url}"
timeoutSeconds: 30
dataProvider: true
checkpointz:
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,4 +31,4 @@ jobs:
with:
consensus: ${{ matrix.consensus }}
network: ${{ matrix.network }}
beacon_node_url: ${{ matrix.network == 'mainnet' && env.MAINNET_BEACON_API_URL || env.HOLESKY_BEACON_API_URL }}
beacon_node_url: ${{ env.MAINNET_BEACON_API_URL }}

0 comments on commit a2585e9

Please sign in to comment.