Skip to content

refactor: split route handlers and keep TraceLayer request logging - #15

Merged
emostov merged 2 commits into
mainfrom
swag/router-handlers-trace
Jun 18, 2026
Merged

emostov merged 2 commits into
mainfrom
swag/router-handlers-trace

Conversation

@emostov

@emostov emostov commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • Move route handler logic and shared state out of router.rs into focused modules.
  • Keep router.rs focused on route wiring.
  • Preserve the existing TraceLayer::new_for_http() request logging layer.

Verification

  • cargo fmt --check
  • cargo test -p helloworld
  • cargo clippy -p helloworld --all-targets -- -D warnings

Notes

  • cargo test --all-targets currently fails on main in crates/e2e because CARGO_BIN_EXE_helloworld is unset; this PR does not change that unrelated harness behavior.

@emostov
emostov marked this pull request as ready for review June 18, 2026 16:55
@emostov
emostov marked this pull request as draft June 18, 2026 16:58
@emostov
emostov marked this pull request as ready for review June 18, 2026 19:01
@emostov
emostov merged commit 3b1c2b1 into main Jun 18, 2026
4 checks passed
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.

2 participants