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

Configurable Dispute Game Parameters #12228

Merged
merged 7 commits into from
Oct 2, 2024

Conversation

blmalone
Copy link
Contributor

@blmalone blmalone commented Oct 1, 2024

For all dispute game parameters, we should make them configurable via op-deployer.

Copy link
Contributor Author

blmalone commented Oct 1, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @blmalone and the rest of your teammates on Graphite Graphite

Copy link
Contributor

semgrep-app bot commented Oct 1, 2024

Semgrep found 8 golang_fmt_errorf_no_params findings:

No fmt.Errorf invocations without fmt arguments allowed

Ignore this finding from golang_fmt_errorf_no_params.

Semgrep found 3 dangerous-exec-command findings:

Detected non-static command inside Command. Audit the input to 'exec.Command'. If unverified user data can reach this call site, this is a code injection vulnerability. A malicious actor can inject a malicious script to execute arbitrary code.

Ignore this finding from dangerous-exec-command.

@blmalone blmalone force-pushed the 10-01-configurable_dispute_game_parameters branch from 3831115 to f1ce841 Compare October 1, 2024 18:34
Base automatically changed from 10-01-configurable_gaslimit_for_op-deployer to develop October 1, 2024 18:55
@blmalone blmalone force-pushed the 10-01-configurable_dispute_game_parameters branch from 716802a to 8b248c9 Compare October 1, 2024 19:28
@blmalone blmalone marked this pull request as ready for review October 1, 2024 19:28
@blmalone blmalone requested review from a team as code owners October 1, 2024 19:28
@blmalone blmalone requested review from mbaxter, protolambda, mds1 and mslipper and removed request for mbaxter and protolambda October 1, 2024 19:28
@blmalone blmalone force-pushed the 10-01-configurable_dispute_game_parameters branch from 6082304 to c26adcf Compare October 2, 2024 13:44
@blmalone blmalone force-pushed the 10-01-configurable_dispute_game_parameters branch from aa21870 to 6fa9a8d Compare October 2, 2024 15:37
@blmalone blmalone added this pull request to the merge queue Oct 2, 2024
Merged via the queue into develop with commit 28283a9 Oct 2, 2024
60 checks passed
@blmalone blmalone deleted the 10-01-configurable_dispute_game_parameters branch October 2, 2024 16:05
protolambda pushed a commit that referenced this pull request Oct 7, 2024
* Configurable Dispute Game Parameters

* fix: more assertions for DeployOPChain.t.sol.

* fix: running pre-pr command.

* fix: changing dispute game types.

* fix: semver lock file change.

* fix: uint64 for maxGameDepth and splitDepth on go side.

* fix: safe casting in solidity
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

Successfully merging this pull request may close these issues.

3 participants