Skip to content

Keep rechecking a missing Dot so a brief ChatGPT 404 doesn't stop the login - #40

Merged
iFixRobots merged 2 commits into
mainfrom
indent/chatgpt-dots-recheck-missing-dot
Nov 5, 2026
Merged

iFixRobots merged 2 commits into
mainfrom
indent/chatgpt-dots-recheck-missing-dot

Conversation

@indent

@indent indent Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

On Nov 4 between 07:50 and 08:20 UTC, five stable accounts on different clusters got HTTP 404 for their Dot room for a few seconds, and the primary-Dot lookup also said there was no Dot:

warn The linked Dot room returned HTTP 404 {"primary_dot_missing":true,"primary_dot_replaced":false}

All five went BAD_CREDENTIALS chatgpt-dot-unavailable and logged nothing more. They only came back about 3 hours later, when a pod restart reconnected them to the same Dot room without the user signing in again. So ChatGPT's answer was temporary, but run treated it as final.

run returned on errDotUnavailable in both the connect loop and the poll loop. Now it's split into connectDot and serve. A missing Dot still sends chatgpt-dot-unavailable and stops polling and the live-signal stream. The client then re-verifies every 5 minutes, and once the Dot room answers again it resumes and goes back to CONNECTED. A Dot that is really gone or replaced keeps the same state as before, and signing in again works the same way.

go build ./..., go vet ./pkg/connector/ and go test ./pkg/... pass.

View in Indent
Tag @indent to continue the conversation here.

@indent

indent Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor Author

Issues

All clear! No issues remaining. 🎉

1 issue already resolved
  • When the Dot goes missing during profile refresh, serve waits 5 minutes before its deferred stream shutdown runs. The live-signal stream keeps running for the whole recheck interval, although the PR intends it to stop. (fixed by commit 4012459)
    Found by Indent Review Agent

CI Checks

All CI checks passed on 4012459.

Review agents

Select any unchecked box below to run or rerun that agent.

Passed (1)
  • Indent Review Agent · No issues found
Full results

Indent Review Agent

  • Summary: No issues found
  • Last ran on commit: 40124598
  • Latest result
    {
      "summary": "No issues found",
      "findings": []
    }

@indent indent Bot added the to be tested label Nov 5, 2026
Comment thread pkg/connector/client.go Resolved
@iFixRobots
iFixRobots merged commit 05f3c68 into main Nov 5, 2026
7 checks passed
@iFixRobots
iFixRobots deleted the indent/chatgpt-dots-recheck-missing-dot branch November 5, 2026 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants