Skip to content

fix(pat): honor server device flow expiry#769

Open
shangguanxuan633-lab wants to merge 3 commits into
DingTalk-Real-AI:mainfrom
shangguanxuan633-lab:codex/dcf-server-expiry-main-20260723
Open

fix(pat): honor server device flow expiry#769
shangguanxuan633-lab wants to merge 3 commits into
DingTalk-Real-AI:mainfrom
shangguanxuan633-lab:codex/dcf-server-expiry-main-20260723

Conversation

@shangguanxuan633-lab

@shangguanxuan633-lab shangguanxuan633-lab commented Jul 23, 2026

Copy link
Copy Markdown
Collaborator

Summary:

  • consume the PAT service expiresInSeconds field
  • use the same effective timeout for CLI display and device-flow polling deadline
  • preserve the legacy 10 minute fallback for missing or invalid values
  • reject and fall back when server expiry exceeds the 30-day DCF protocol/storage upper bound; this is input validation, not a second client-owned TTL

Tests:

  • focused expiry resolver and handler tests
  • DWS_PACKAGE_VERSION=0.0.0-test go test ./internal/app -count=1
  • DWS_PACKAGE_VERSION=0.0.0-test go test ./...
  • go build -o /tmp/dws-dcf-expiry-cli-build ./cmd

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.

1 participant