Skip to content

Commit 644000d

Browse files
committed
aws: remove http.DefaultClient
Signed-off-by: Hank Donnay <[email protected]>
1 parent 95bfa5b commit 644000d

File tree

1 file changed

+0
-4
lines changed

1 file changed

+0
-4
lines changed

aws/updater.go

-4
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,6 @@ type Updater struct {
2929
func NewUpdater(release Release) (*Updater, error) {
3030
return &Updater{
3131
release: release,
32-
c: http.DefaultClient, // TODO(hank) Remove DefaultClient
3332
}, nil
3433
}
3534

@@ -45,9 +44,6 @@ func (u *Updater) Configure(ctx context.Context, _ driver.ConfigUnmarshaler, c *
4544

4645
func (u *Updater) Fetch(ctx context.Context, fingerprint driver.Fingerprint) (io.ReadCloser, driver.Fingerprint, error) {
4746
ctx = zlog.ContextWithValues(ctx, "component", "aws/Updater.Fetch")
48-
if u.c == http.DefaultClient { // OK: checking for log purposes
49-
zlog.Warn(ctx).Msg("DefaultClient used, this is almost certainly wrong")
50-
}
5147
client, err := NewClient(ctx, u.c, u.release)
5248
if err != nil {
5349
return nil, "", fmt.Errorf("failed to create client: %v", err)

0 commit comments

Comments
 (0)