mobile_nebula icon indicating copy to clipboard operation
mobile_nebula copied to clipboard

switch to LongPollWait

Open JackDoan opened this issue 2 months ago • 1 comments

JackDoan avatar Dec 03 '25 18:12 JackDoan

I think the most recent commit updated the timeout from 30 seconds to 60 seconds - did you test to see if this was long enough when a LongPollWait doesn't immediately return?

I ask because 60 seconds looks likely to race with the API handler to me, which also uses a 60 second timeout beginning when the request is received (so not including standing up the connection, etc.): https://github.com/DefinedNet/api/blob/b09af0ef4fd2e548d18d61283aef014716b4eb88/internal/app/dnclient_api.go#L577

johnmaguire avatar Dec 23 '25 14:12 johnmaguire