frontend Arbitrum support not working

Description

Setting NEXT_PUBLIC_ROLLUP_TYPE: "arbitrum" prevents the startup of the application

🌀 Checking environment variables and their placeholders congruity...
👍 All good!

🌀 Validating ENV variables values...
🚨 ENVs validation failed with the following errors:
     NEXT_PUBLIC_ROLLUP_L2_WITHDRAWAL_URL cannot not be used if NEXT_PUBLIC_ROLLUP_TYPE is not defined

Link to the page

asdf

Steps to reproduce

Go to '...'
Click on '...'
Scroll down to '...'
See error

App version

v1.32.0

Browser

No response

Operating system

None

Additional information

No response

Jul 05 '24 06:07 InoMurko

If I set it to ``` NEXT_PUBLIC_ROLLUP_TYPE: "optimistic"

it works

Jul 05 '24 06:07 InoMurko

/app $ env | grep NEXT_PUBLIC_ROLLUP_TYPE
NEXT_PUBLIC_ROLLUP_TYPE=arbitrum

and removing NEXT_PUBLIC_ROLLUP_L2_WITHDRAWAL_URL works.

so the combination of these two variables breaks the application

NEXT_PUBLIC_ROLLUP_L2_WITHDRAWAL_URL="whatever"
NEXT_PUBLIC_ROLLUP_TYPE=arbitrum

Jul 05 '24 07:07 InoMurko

In the documentation here you can see that NEXT_PUBLIC_ROLLUP_L2_WITHDRAWAL_URL can be used only with 'optimistic' type of rollup. So you don't need NEXT_PUBLIC_ROLLUP_L2_WITHDRAWAL_URL variable for arbitrum L2 and can just remove it from your envs list

Jul 05 '24 10:07 isstuev

Actually, it says it is REQUIRED for optimistic rollup types, not that it is forbidden for Arbitrum support (or any other) should simply ignore this variable or print a warning because currently if it's set it prevents frontend app to start.

Jul 05 '24 10:07 InoMurko

I'm changing the error message, but we won't ignore unused variables, we're trying to keep a minimal variable list to minimize payload and prevent errors and misunderstandings

Jul 05 '24 10:07 isstuev