(feat) add retry and fail logic with configurable timeout and maximum…

[ozkanturidi]

(feat) add retry and fail logic for bootstrap keeper with configurable timeout and maximum number of retries variables

[v0lkan]

Realized a blocking issue if the api request never succeeds; recommended possible solution.

Feel free to take it, update it, mutate it.

Thanks for your contribution.

[v0lkan]

nit: can we limit the line lenght to 80 chars pls.

[v0lkan]

If this request hangs:

1. The keeperCtx deadline expires after timeout
2. The context's Done() channel gets closed
3. Nothing happens — because nothing is listening to that channel
4. api.Contribute continues blocking forever

Suggestion: wrap this in a goroutine:

func contributeWithContext(ctx context.Context, api *spike.API, share []byte, keeperID string) error {
    done := make(chan error, 1)
    
    go func() {
        err := api.Contribute(share, keeperID)
        done <- err
    }()
    
    select {
    case err := <-done:
        return err
    case <-ctx.Done():
        return ctx.Err()
    }
}

and

err := contributeWithContext(keeperCtx, api, keeperShare, keeperID)

[v0lkan]

@ozkanturidi lint an unit tests are failing. — can you check them out?

Don't mind integration tests, it's a different bug; but lint and units need to pass.