Skip to content

Commit

Permalink
signalmeow: only apply provisioning timeout to websocket step
Browse files Browse the repository at this point in the history
  • Loading branch information
tulir committed Aug 10, 2024
1 parent efe8098 commit 6f4781f
Showing 1 changed file with 4 additions and 4 deletions.
8 changes: 4 additions & 4 deletions pkg/signalmeow/provisioning.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,9 +85,9 @@ func PerformProvisioning(ctx context.Context, deviceStore store.DeviceStore, dev
go func() {
defer close(c)

ctx, cancel := context.WithTimeout(ctx, 2*time.Minute)
timeoutCtx, cancel := context.WithTimeout(ctx, 2*time.Minute)
defer cancel()
ws, resp, err := web.OpenWebsocket(ctx, web.WebsocketProvisioningPath)
ws, resp, err := web.OpenWebsocket(timeoutCtx, web.WebsocketProvisioningPath)
if err != nil {
log.Err(err).Any("resp", resp).Msg("error opening provisioning websocket")
c <- ProvisioningResponse{State: StateProvisioningError, Err: err}
Expand All @@ -96,15 +96,15 @@ func PerformProvisioning(ctx context.Context, deviceStore store.DeviceStore, dev
defer ws.Close(websocket.StatusInternalError, "Websocket StatusInternalError")
provisioningCipher := NewProvisioningCipher()

provisioningURL, err := startProvisioning(ctx, ws, provisioningCipher)
provisioningURL, err := startProvisioning(timeoutCtx, ws, provisioningCipher)
if err != nil {
log.Err(err).Msg("startProvisioning error")
c <- ProvisioningResponse{State: StateProvisioningError, Err: err}
return
}
c <- ProvisioningResponse{State: StateProvisioningURLReceived, ProvisioningURL: provisioningURL, Err: err}

provisioningMessage, err := continueProvisioning(ctx, ws, provisioningCipher)
provisioningMessage, err := continueProvisioning(timeoutCtx, ws, provisioningCipher)
if err != nil {
log.Err(err).Msg("continueProvisioning error")
c <- ProvisioningResponse{State: StateProvisioningError, Err: err}
Expand Down

0 comments on commit 6f4781f

Please sign in to comment.