Skip to content

Sync EdgeZero PR #257 updates#761

Draft
aram356 wants to merge 1 commit into
mainfrom
chore/sync-edgezero-pr-257
Draft

Sync EdgeZero PR #257 updates#761
aram356 wants to merge 1 commit into
mainfrom
chore/sync-edgezero-pr-257

Conversation

@aram356

@aram356 aram356 commented Jun 10, 2026 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Summary

  • update EdgeZero git dependencies to Enable strict clippy with documented allow-list and defensive-coding pass stackpop/edgezero#257 head commit e78871407057c392d78fa4a0bcec5e4ca8ba80b5
  • sync EdgeZero tool pins: Rust 1.95.0, Fastly CLI 15.1.0, Node 24.12.0, Viceroy 0.17.0, and Wasmtime 44.0.1
  • update Fastly/log-fastly to 0.12 and adapt Fastly TLS getter handling for the 0.12 API
  • keep the strict workspace clippy posture, reduce local allows where refactoring was practical, and document the remaining workspace allow
  • keep all crate package names and directories on the trusted-server-* naming scheme, with references updated

Verification

  • cargo --version / rustc --version
  • fastly version
  • viceroy --version
  • cargo fmt --all -- --check
  • git diff --check
  • scripts/check-integration-dependency-versions.sh
  • cargo check --workspace --all-targets --all-features
  • cargo clippy --workspace --all-targets --all-features -- -D warnings
  • cargo test --workspace
  • cargo build --package trusted-server-adapter-fastly --release --target wasm32-wasip1
  • cargo check --manifest-path crates/trusted-server-openrtb-codegen/Cargo.toml
  • cargo check --manifest-path crates/trusted-server-integration-tests/Cargo.toml
  • cd crates/trusted-server-js/lib && npx vitest run

References stackpop/edgezero#257

@aram356 aram356 marked this pull request as draft June 10, 2026 06:10
@aram356 aram356 self-assigned this Jun 10, 2026
@aram356 aram356 force-pushed the chore/sync-edgezero-pr-257 branch from f078d3b to 396d270 Compare June 10, 2026 06:11
github-advanced-security[bot]
github-advanced-security AI found potential problems Jun 10, 2026
View reviewed changes
Comment thread crates/trusted-server-core/src/backend.rs
"INSECURE: certificate check disabled for backend: {}",
backend_name
);
log::warn!("INSECURE: certificate check disabled for backend: {backend_name}");
Comment thread crates/trusted-server-core/src/backend.rs
log::warn!("INSECURE: certificate check disabled for backend: {backend_name}");
}
log::info!("enable ssl for backend: {}", backend_name);
log::info!("enable ssl for backend: {backend_name}");
Comment thread crates/trusted-server-core/src/backend.rs
backend_name,
host_with_port
);
log::info!("created dynamic backend: {backend_name} -> {host_with_port}");
Comment thread crates/trusted-server-core/src/backend.rs
let msg = e.to_string();
if msg.contains("NameInUse") || msg.contains("already in use") {
log::info!("reusing existing dynamic backend: {}", backend_name);
log::info!("reusing existing dynamic backend: {backend_name}");
@aram356 aram356 force-pushed the chore/sync-edgezero-pr-257 branch 3 times, most recently from 41e7268 to b6c4a79 Compare June 10, 2026 19:09
@aram356 aram356 force-pushed the chore/sync-edgezero-pr-257 branch from b6c4a79 to 159c731 Compare June 11, 2026 08:04
@aram356 aram356 changed the title Sync EdgeZero PR #257 strict clippy updates Sync EdgeZero PR #257 updates Jun 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@github-advanced-security github-advanced-security[bot] github-advanced-security[bot] left review comments

Reviewers whose approvals may not affect merge requirements

At least 1 approving review is required to merge this pull request.

Assignees

@aram356 aram356

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@aram356 @github-advanced-security