Skip to content

Commit 85c7f96

Browse files
authored
Merge pull request #376 from defang-io/eric-delete-config-refactor
Implement use of DeleteSecrets cli call.
2 parents 307eab1 + d6de496 commit 85c7f96

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

src/cmd/cli/command/commands.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@ func prettyError(err error) error {
4646
// To avoid printing the internal gRPC error code
4747
var cerr *connect.Error
4848
if errors.As(err, &cerr) {
49-
term.Debug(" - Server error:", err)
50-
err = errors.Unwrap(err)
49+
term.Debug(" - Server error:", cerr)
50+
err = errors.Unwrap(cerr)
5151
}
5252
return err
5353

src/pkg/cli/client/grpc.go

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -125,10 +125,8 @@ func (g GrpcClient) PutConfig(ctx context.Context, req *defangv1.SecretValue) er
125125
func (g GrpcClient) DeleteConfig(ctx context.Context, req *defangv1.Secrets) error {
126126
// _, err := g.client.DeleteSecrets(ctx, &connect.Request[v1.Secrets]{Msg: req}); TODO: implement this in the server
127127
var errs []error
128-
for _, name := range req.Names {
129-
_, err := g.client.PutSecret(ctx, &connect.Request[defangv1.SecretValue]{Msg: &defangv1.SecretValue{Name: name}})
130-
errs = append(errs, err)
131-
}
128+
_, err := g.client.DeleteSecrets(ctx, &connect.Request[defangv1.Secrets]{Msg: &defangv1.Secrets{Names: req.Names}})
129+
errs = append(errs, err)
132130
return errors.Join(errs...)
133131
}
134132

0 commit comments

Comments
 (0)