fix: ecdsa recovery failure may cause executor to panic #6182
Triggered via pull request
November 19, 2024 21:16
nhtyy
Status
Failure
Total duration
36m 34s
Artifacts
–
pr.yml
on: pull_request
Test (x86-64)
33m 12s
Test (ARM)
35m 50s
Formatting & Clippy
1m 11s
Examples
2m 27s
Tests
2m 17s
CLI
9m 6s
Performance
6m 16s
Spell Check
9s
Matrix: Low Memory
Annotations
6 errors and 12 warnings
|
Spell Check
HttpError: Validation Failed: {"resource":"PullRequestReviewComment","code":"invalid","field":"pull_request_review_thread.path"}, {"resource":"PullRequestReviewComment","code":"missing_field","field":"pull_request_review_thread.diff_hunk"}
|
|
Spell Check
HttpError: Validation Failed: {"resource":"PullRequestReviewComment","code":"invalid","field":"pull_request_review_thread.path"}, {"resource":"PullRequestReviewComment","code":"missing_field","field":"pull_request_review_thread.diff_hunk"}
|
|
Spell Check
Error: There were some changed files, please update your PR with the code review suggestions
|
|
Formatting & Clippy
The process '/home/runner/.cargo/bin/cargo' failed with exit code 2
|
|
Examples
Process completed with exit code 2.
|
|
Tests
Process completed with exit code 2.
|
|
Spell Check
The following actions uses node12 which is deprecated and will be forced to run on node16: getsentry/action-git-diff-suggestions@main. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
|
Spell Check
The following actions use a deprecated Node.js version and will be forced to run on node20: getsentry/action-git-diff-suggestions@main. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Test (x86-64):
crates/core/executor/src/hook.rs#L83
use of deprecated function `hook::hook_ecrecover`: Use `hook_ecrecover_v2` instead.
|
|
Test (x86-64)
`sp1-core-executor` (lib test) generated 1 warning (1 duplicate)
|
|
Test (x86-64):
crates/core/executor/src/hook.rs#L83
use of deprecated function `hook::hook_ecrecover`: Use `hook_ecrecover_v2` instead.
|
|
Test (x86-64)
`sp1-core-executor` (lib) generated 1 warning
|
|
Test (x86-64)
`sp1-core-executor` (lib test) generated 1 warning (1 duplicate)
|
|
Test (ARM):
crates/core/executor/src/hook.rs#L83
use of deprecated function `hook::hook_ecrecover`: Use `hook_ecrecover_v2` instead.
|
|
Test (ARM)
`sp1-core-executor` (lib test) generated 1 warning (1 duplicate)
|
|
Test (ARM):
crates/core/executor/src/hook.rs#L83
use of deprecated function `hook::hook_ecrecover`: Use `hook_ecrecover_v2` instead.
|
|
Test (ARM)
`sp1-core-executor` (lib) generated 1 warning
|
|
Test (ARM)
`sp1-core-executor` (lib test) generated 1 warning (1 duplicate)
|