fix(GRO-2652): reset authInitPromise on failure so retry loop actually retries #225

Merged

Flea Flicker merged 2 commits from fix/GRO-2652-auth-promise-reset into dev 2026-08-05 10:00:13 +00:00

Conversation 0 Commits 2 Files Changed 1

+6 -1

2 Commits

Author	SHA1	Message	Date
Flea Flicker	a1b27b5501	fix(GRO-2652): reset authInitPromise on failure so retry loop actually retries ... Previously the initAuth() function set authInitPromise to the async IIFE's Promise but never cleared it on rejection. The index.ts retry loop called initAuth() up to 10 times, but on attempt 2+ the check `if (authInitPromise) { await authInitPromise; return; }` would immediately re-throw the original rejection without performing a real retry. Fix: wrap the final `await authInitPromise` in try/catch and reset authInitPromise = null on error. This allows the index.ts retry loop to create a fresh attempt on each call after a failure. Co-Authored-By: Paperclip <noreply@paperclip.ing>	2026-08-05 09:52:31 +00:00
Flea Flicker	ae0ce3824f	fix(GRO-2652): reset authInitPromise on failure so retry loop actually retries ... Previously the initAuth() function set authInitPromise to the async IIFE's Promise but never cleared it on rejection. The index.ts retry loop called initAuth() up to 10 times, but on attempt 2+ the check `if (authInitPromise) { await authInitPromise; return; }` would immediately re-throw the original rejection without performing a real retry. Fix: wrap the final `await authInitPromise` in try/catch and reset authInitPromise = null on error. This allows the index.ts retry loop to create a fresh attempt on each call after a failure. Co-Authored-By: Paperclip <noreply@paperclip.ing>	2026-08-05 09:44:12 +00:00