Skip to content

Tech-debt batch 20: follow-ups — admin refusal reasons, OAuth refusal page, eager outbox serialization, refresh-family max age, token purge on user delete, bearer /me (#653–#658) - #671

Merged
antosubash merged 7 commits into
mainfrom
fix/debt-batch-20
Oct 24, 2026
Merged

antosubash merged 7 commits into
mainfrom
fix/debt-batch-20

Conversation

@antosubash

Copy link
Copy Markdown
Owner

Summary

Closes #653
Closes #654
Closes #655
Closes #656
Closes #657
Closes #658

Verification

  • Browser QA on port 8220, all five scenarios clean:
    • The OAuth error codes show their messages, and an injected code shows nothing.
    • Self-demote shows the translated reason.
    • Bearer /me returns 200, and 401 without credentials.
    • Deleting a user purges their tokens once the outbox worker runs.
    • The new setting appears in /admin/settings.
  • Local CI: make lint, make test-py (6330 passed) and make test-js (835 passed) are green.
  • Verification report: https://claude.ai/artifact/XbdnMrrwXeELVERk9reUtC

QA Report

Report links are Claude Artifacts and start private — the author can share them
from claude.ai/code/artifacts if reviewers need access.

Test plan

  • A refused OAuth sign-in lands on the login page with a readable message
  • CI is green

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 24, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ✅ Completed 2026-10-24T20:42:32.189574Z a0f5db6 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying simple-module-python with  Cloudflare Pages  Cloudflare Pages

Latest commit: a0f5db6
Status: ✅  Deploy successful!
Preview URL: https://9630d3d7.simple-module-python.pages.dev
Branch Preview URL: https://fix-debt-batch-20.simple-module-python.pages.dev

View logs

@antosubash
antosubash merged commit 3023bfc into main Oct 24, 2026
17 checks passed
@antosubash
antosubash deleted the fix/debt-batch-20 branch October 24, 2026 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment