-
Notifications
You must be signed in to change notification settings - Fork 0
feat: add request logging and admin audit foundations #48
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
Merged
Changes from all commits
Commits
Show all changes
16 commits
Select commit
Hold shift + click to select a range
045e04d
feat: add request logging and admin audit foundations for issue 7
wauputr4 13d739f
fix: resolve clippy failures for issue 7 logging changes
wauputr4 29b7b04
fix: align stream error logging and request log conversion handling
wauputr4 dc8ebb6
docs: enforce OpenAI-compatible contract across provider types
wauputr4 f8facd5
scripts: add OpenAI-compatible model sync helper
wauputr4 dcaa729
test: add upstream model sync to alpha smoke
wauputr4 5a194b1
test: harden alpha smoke wait and proof notes
wauputr4 1ac30a5
fix: address actionable PR review feedback
wauputr4 98ebf2c
fix: make gateway request log async
wauputr4 7e32c72
fix: tighten release review follow-ups
wauputr4 18e5932
docs: clarify model sync python dependency
wauputr4 ef8b393
docs: clarify alpha readiness around issue 51
wauputr4 e85316a
feat: add openai-compatible responses endpoint
wauputr4 4afbc48
style: format gateway helpers
wauputr4 4644b09
fix: satisfy clippy in gateway helpers
wauputr4 6d9f8d0
docs: align alpha readiness logging boundary
wauputr4 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When the
admin_audit_logsinsert fails here (for example because migrations were disabled or the audit table/schema is unavailable), the credit grant has already committed and this handler still returns success, leaving a balance-changing admin action with no audit row. Since the audit log is being introduced as durable admin audit coverage, record it in the same transaction as the credit ledger/wallet update or fail the mutation when the audit write cannot be persisted.Useful? React with 👍 / 👎.