Skip to content

Commit

Permalink
dkg: remove todos (#2900)
Browse files Browse the repository at this point in the history
Removes TODOs in the `dkg` package.

category: misc
ticket: none
  • Loading branch information
xenowits authored Feb 26, 2024
1 parent 2676a14 commit 7d0d0a7
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
1 change: 0 additions & 1 deletion dkg/disk.go
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,6 @@ func writeKeysToKeymanager(ctx context.Context, keymanagerURL, authToken string,
}
passwords = append(passwords, password)

// TODO(gsora): needs to go away once we get rid of kryptology
store, err := keystore.Encrypt(s.SecretShare, password, rand.Reader)
if err != nil {
return err
Expand Down
2 changes: 1 addition & 1 deletion dkg/sync/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -179,7 +179,7 @@ func (c *Client) sendMsgs(ctx context.Context, stream network.Stream) (relayBrok
resp, err := c.sendMsg(stream, shutdown)
if isRelayError(err) {
return true, false, err // Reconnect on relay errors
} else if err != nil { // TODO(dhruv): differentiate between connection errors and other errors.
} else if err != nil {
return false, true, err
} else if shutdown {
return false, false, nil
Expand Down

0 comments on commit 7d0d0a7

Please sign in to comment.