Skip to content

refactor(core): default gateway config#2457

Draft
elezar wants to merge 1 commit into
mainfrom
codex/config-default-trait
Draft

refactor(core): default gateway config#2457
elezar wants to merge 1 commit into
mainfrom
codex/config-default-trait

Conversation

@elezar

@elezar elezar commented Jul 24, 2026

Copy link
Copy Markdown
Member

Summary

Add a Default implementation for the programmatic gateway Config so no-TLS default construction can use the standard Rust trait. Update existing no-TLS test/setup call sites from Config::new(None) to Config::default().

Related Issue

N/A

Changes

  • Added impl Default for Config delegating to Config::new(None).
  • Updated core and server test/support call sites to use Config::default() where they want the standard gateway defaults.

Testing

  • /home/elezar/.cargo/bin/cargo fmt
  • /home/elezar/.cargo/bin/cargo test -p openshell-core config::tests:: --lib passes
  • /home/elezar/.cargo/bin/cargo test -p openshell-server configured_compute_driver --lib attempted, blocked by local z3-sys/z3.h header availability
  • /home/elezar/.cargo/bin/cargo test -p openshell-server grpc_rate_limit --lib attempted, blocked by local z3-sys/z3.h header availability
  • /home/elezar/.cargo/bin/cargo test -p openshell-server configured_compute_driver --lib --features bundled-z3 attempted, blocked by bindgen missing local C header stdbool.h
  • /home/elezar/.cargo/bin/cargo test -p openshell-server grpc_rate_limit --lib --features bundled-z3 attempted, blocked by bindgen missing local C header stdbool.h
  • E2E tests added/updated (not applicable)

Checklist

  • Follows Conventional Commits
  • Commits are signed off (DCO)

Signed-off-by: Evan Lezar <elezar@nvidia.com>
@copy-pr-bot

copy-pr-bot Bot commented Jul 24, 2026

Copy link
Copy Markdown

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

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.

1 participant