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

Infer other addresses from SystemConfig once MCP L1 is launched #71

Open
zchn opened this issue Jan 30, 2024 · 3 comments
Open

Infer other addresses from SystemConfig once MCP L1 is launched #71

zchn opened this issue Jan 30, 2024 · 3 comments

Comments

@zchn
Copy link
Contributor

zchn commented Jan 30, 2024

          The goal of this file to serve as a reference and source of truth, which we verify in the `CheckSecurityConfigs` script? I ask because as of https://github.com/ethereum-optimism/optimism/pull/6555, we can now only define a chain's `SystemConfig` and then discover everything else. Wondering if we can leverage that in this repo (once everyone is on MCP L1 contracts)

Originally posted by @mds1 in #69 (comment)

@geoknee
Copy link
Collaborator

geoknee commented Jul 8, 2024

The downside of this is that the other addresses will no longer be tracked in the repository, and therefore not inspectable by humans / services / software packages that want to read them.

@mds1
Copy link
Contributor

mds1 commented Jul 28, 2024

@geoknee What do you mean exactly? We can, for example, only ask for their SystemConfig address if it simplifies the add chain process, but still generate output files that contain all other system addresses for humans / services / software packages that want to read them.

@geoknee
Copy link
Collaborator

geoknee commented Aug 6, 2024

Yeah you're right that should work well. I think I was skipping all the way to "only store the SystemConfig address" which is not what this ticket is about.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants