Skip to content

fix(auth): adapt CIMD redirects for Workers - #845

Merged
tiankaima merged 1 commit into
mainfrom
agent/fix-cimd-worker-redirect
Aug 14, 2026
Merged

fix(auth): adapt CIMD redirects for Workers#845
tiankaima merged 1 commit into
mainfrom
agent/fix-cimd-worker-redirect

Conversation

@tiankaima

Copy link
Copy Markdown
Member

Summary

  • translate Better Auth CIMD redirect:error requests to Workers-supported redirect:manual
  • preserve fail-closed redirect handling because CIMD accepts only status 200
  • add a transport regression test that preserves headers and abort signals

Validation

  • bunx vitest run tests/unit/cimd-fetch-policy.test.ts tests/unit/oauth-cimd-plugin.test.ts tests/unit/wrangler-rate-limit-config.test.ts
  • bunx biome check src/lib/auth/cimd-fetch-policy.ts tests/unit/cimd-fetch-policy.test.ts
  • bunx tsc --noEmit -p tsconfig.typecheck.json
  • bunx tsc --noEmit -p tsconfig.typecheck.tests.json
  • bunx wrangler types --include-runtime=false --check
  • bun run build

@tiankaima
tiankaima marked this pull request as ready for review August 14, 2026 16:19
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
life-ustc 58a9892 Aug 14 2026, 04:20 PM

@github-actions

Copy link
Copy Markdown

@tiankaima
tiankaima merged commit b77a13d into main Aug 14, 2026
17 checks passed
@tiankaima
tiankaima deleted the agent/fix-cimd-worker-redirect branch August 14, 2026 16:33
github-actions Bot pushed a commit that referenced this pull request Aug 14, 2026
## [1.90.4](v1.90.3...v1.90.4) (2026-08-14)

### Bug Fixes

* **auth:** adapt CIMD redirects for Workers ([#845](#845)) ([b77a13d](b77a13d))
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 1.90.4 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant