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

Tidy Eth2Config generation at runtime #602

Closed
paulhauner opened this issue Nov 15, 2019 · 1 comment · Fixed by #912
Closed

Tidy Eth2Config generation at runtime #602

paulhauner opened this issue Nov 15, 2019 · 1 comment · Fixed by #912

Comments

@paulhauner
Copy link
Member

Description

Presently, when the lighthouse binary is started an Environment is created which contains an Eth2Config.

Then, this code ignores the environment.eth2_spec and replaces the Eth2Config in it's local context.

This means that there's no guarantee of consistency of the Eth2Config from an Environment.

Steps to resolve

Notes

This issue is only valid after #542 is merged.

@paulhauner paulhauner added this to To-Do (Nice-to-Have) in Security Review Mar 2, 2020
@ackintosh
Copy link
Member

Hi, I'm addressing this issue, will file a PR to close this. 😃

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
Security Review
To-Do (Nice-to-Have)
Development

Successfully merging a pull request may close this issue.

2 participants