feat: propagate audit correlation ids - #14
Conversation
seonghobae
left a comment
There was a problem hiding this comment.
Current-head review request for 9d6d33a. Please independently verify header parsing and 400 behavior, propagation into the PR13 audit rows, generated-ID fallback, and exact-head workflow assertions. This is a COMMENT only; no approval or protected-merge bypass is requested.
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
seonghobae
left a comment
There was a problem hiding this comment.
Current-head review request for c75ff72. Re-verify the exact-head diff, required Checks, tenant/security boundaries, and any valid review findings after the shared Rust component-install fix. COMMENT only; no approval or protected-merge bypass is requested.
Summary
X-Correlation-IDUUID on assessment import, completion publication, credential issue, and credential revocationEvidence
cargo test --workspace --all-targets --lockedcargo clippy --workspace --all-targets --locked -- -D warningsRUSTDOCFLAGS=-Dwarnings cargo doc --workspace --no-deps --lockedcargo build --bin lms_api --lockedcargo fmt --all -- --checkactionlintgit diff --checkGate
This is stacked on PR #13 and remains unmerged product evidence until an independent current-head review and required Checks pass. Merge only after re-fetching exact head
9d6d33aand confirming protected rules.