Skip to content

DRAFT: OpenRTB full implementation#390

Draft
ChristianPavilonis wants to merge 3 commits intomainfrom
feature/openrtb-full-implementation
Draft

DRAFT: OpenRTB full implementation#390
ChristianPavilonis wants to merge 3 commits intomainfrom
feature/openrtb-full-implementation

Conversation

@ChristianPavilonis
Copy link
Collaborator

Summary

Changes

File Change

Closes

Closes #

Test plan

  • cargo test --workspace
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo fmt --all -- --check
  • JS tests: cd crates/js/lib && npx vitest run
  • JS format: cd crates/js/lib && npm run format
  • Docs format: cd docs && npm run format
  • WASM build: cargo build --bin trusted-server-fastly --release --target wasm32-wasip1
  • Manual testing via fastly compute serve
  • Other:

Checklist

  • Changes follow CLAUDE.md conventions
  • No unwrap() in production code — use expect("should ...")
  • Uses tracing macros (not println!)
  • New code has tests
  • No secrets or credentials committed

@ChristianPavilonis ChristianPavilonis linked an issue Mar 2, 2026 that may be closed by this pull request
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.

Complete OpenRTB implementation

1 participant