Fix #439: Api key creation not working - #440
Open
ekwav-agent wants to merge 3 commits into
Open
Conversation
Ekwav
requested changes
Aug 27, 2026
| + "WITH default_time_to_live = 15552000"); // 180 days | ||
|
|
||
| // Ensure secondary index on user_id for querying by user | ||
| session.Execute("CREATE INDEX IF NOT EXISTS ON api_keys (user_id)"); |
Ekwav
requested changes
Aug 27, 2026
Ekwav
left a comment
Ekwav
left a comment
Member
There was a problem hiding this comment.
There is now no behavioral change anymore, I don't think virtual is a valid change, why was that made
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
Automated patch for #439 from task
task_jt4lpfe3l5p7l5cfjm7q.Base branch:
mainBase commit:
68aa16ce4929f7adfbf350546b23bcfa0a9165ceReview: separate codex session recorded.
Tests:
container_build— sha256:ed6b413312081d47c9af7fcacddf3c3fb6ded0eee4a49cdf7628a7bafa9c7b12regression_base_fail_patch_pass— trusted-harness exact command, isolated checkout servers, and reviewed test overlays: overlay_derivation=applied:1 sibling_setup_exit=0 overlay_setup_exit=0 overlay_review_digest=unavailable overlay_receipt_sha256=unavailable base_setup_exit=0 base_setup_log_sha256=unavailable server_isolation_exit=0 base_exit=1 base_log_sha256=7678f7d9b6a06d461f3f1cafe24e0f06c341e9ccb237733248f4ef3e26a48367 patched_exit=0 patched_log_sha256=426c99ca80c2d44f6a9e9f81b6d38f6cd2a35e4de8bf9b10eeb0f84955a7eb30Corrected the credential regression.
/cofl apibehavior unchanged: existing active keys are displayed; a key is generated only when none exist.Validation:
68aa16ce: regression fails the intended assertion because the query lacksALLOW FILTERING; 261 passed, 2 skipped, 1 failed.docker build --pull --tag coflnet-sky-mod-commands-test .passes.git diff HEAD --checkpasses..coflnet-regression.jsonwith the exact comparison command.Production report
e9a3c2supports that the failing input was bare/cofl api; it does not expose the underlying database exception, so no stronger telemetry claim is made. Auction data in that report was unrelated and was not fetched. Authentication, payments, player data, permissions, and report tracing remain unchanged.This PR cannot be merged or approved by the automation identity; human review is required.