Skip to content

fix: allow dashboard refresh cookie on http deployments#5026

Open
haosenwang1018 wants to merge 1 commit intomem0ai:mainfrom
haosenwang1018:fix/dashboard-refresh-cookie-secure
Open

fix: allow dashboard refresh cookie on http deployments#5026
haosenwang1018 wants to merge 1 commit intomem0ai:mainfrom
haosenwang1018:fix/dashboard-refresh-cookie-secure

Conversation

@haosenwang1018
Copy link
Copy Markdown

Summary

  • derive the dashboard refresh cookie secure flag from DASHBOARD_URL when it is configured
  • keep secure cookies for HTTPS deployments and production deployments without an explicit dashboard URL
  • allow documented self-hosted HTTP/IP deployments to retain the refresh cookie after login

Fixes #5024

Tests

  • pnpm exec prettier --write src/app/api/auth/refresh/route.ts
  • pnpm run typecheck
  • pnpm exec prettier --check src/app/api/auth/refresh/route.ts

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Unable to access the Dashboard

1 participant