From 4e8b2beea1e4935d4552e031b4367dfc731187e5 Mon Sep 17 00:00:00 2001 From: Josh Humphries <2035234+jhump@users.noreply.github.com> Date: Tue, 9 Apr 2024 17:14:15 -0400 Subject: [PATCH] make staticcheck happy --- desc_source.go | 4 ++-- desc_source_test.go | 4 ++-- grpcurl.go | 5 ++--- internal/testing/cmd/bankdemo/main.go | 5 ++--- 4 files changed, 8 insertions(+), 10 deletions(-) diff --git a/desc_source.go b/desc_source.go index 03caaf1b..d12fae0d 100644 --- a/desc_source.go +++ b/desc_source.go @@ -5,7 +5,7 @@ import ( "errors" "fmt" "io" - "io/ioutil" + "os" "sync" "github.com/golang/protobuf/proto" //lint:ignore SA1019 we have to import this because it appears in exported API @@ -41,7 +41,7 @@ type DescriptorSource interface { func DescriptorSourceFromProtoSets(fileNames ...string) (DescriptorSource, error) { files := &descriptorpb.FileDescriptorSet{} for _, fileName := range fileNames { - b, err := ioutil.ReadFile(fileName) + b, err := os.ReadFile(fileName) if err != nil { return nil, fmt.Errorf("could not load protoset file %q: %v", fileName, err) } diff --git a/desc_source_test.go b/desc_source_test.go index a40d6aff..b043b6d8 100644 --- a/desc_source_test.go +++ b/desc_source_test.go @@ -2,7 +2,7 @@ package grpcurl import ( "bytes" - "io/ioutil" + "os" "testing" "github.com/golang/protobuf/proto" //lint:ignore SA1019 we have to import this because it appears in exported API @@ -34,7 +34,7 @@ func TestWriteProtoset(t *testing.T) { } func loadProtoset(path string) (*descriptorpb.FileDescriptorSet, error) { - b, err := ioutil.ReadFile(path) + b, err := os.ReadFile(path) if err != nil { return nil, err } diff --git a/grpcurl.go b/grpcurl.go index c609f47c..fb298c35 100644 --- a/grpcurl.go +++ b/grpcurl.go @@ -14,7 +14,6 @@ import ( "encoding/base64" "errors" "fmt" - "io/ioutil" "net" "os" "regexp" @@ -545,7 +544,7 @@ func ClientTLSConfig(insecureSkipVerify bool, cacertFile, clientCertFile, client } else if cacertFile != "" { // Create a certificate pool from the certificate authority certPool := x509.NewCertPool() - ca, err := ioutil.ReadFile(cacertFile) + ca, err := os.ReadFile(cacertFile) if err != nil { return nil, fmt.Errorf("could not read ca certificate: %v", err) } @@ -582,7 +581,7 @@ func ServerTransportCredentials(cacertFile, serverCertFile, serverKeyFile string if cacertFile != "" { // Create a certificate pool from the certificate authority certPool := x509.NewCertPool() - ca, err := ioutil.ReadFile(cacertFile) + ca, err := os.ReadFile(cacertFile) if err != nil { return nil, fmt.Errorf("could not read ca certificate: %v", err) } diff --git a/internal/testing/cmd/bankdemo/main.go b/internal/testing/cmd/bankdemo/main.go index 7a73864e..02bf635f 100644 --- a/internal/testing/cmd/bankdemo/main.go +++ b/internal/testing/cmd/bankdemo/main.go @@ -7,7 +7,6 @@ import ( "encoding/json" "flag" "fmt" - "io/ioutil" "net" "os" "os/signal" @@ -130,7 +129,7 @@ type svr struct { } func (s *svr) load() error { - accts, err := ioutil.ReadFile(s.datafile) + accts, err := os.ReadFile(s.datafile) if err != nil && !os.IsNotExist(err) { return err } @@ -162,7 +161,7 @@ func (s *svr) flush() { if b, err := json.Marshal(accounts); err != nil { grpclog.Errorf("failed to save data to %q", s.datafile) - } else if err := ioutil.WriteFile(s.datafile, b, 0666); err != nil { + } else if err := os.WriteFile(s.datafile, b, 0666); err != nil { grpclog.Errorf("failed to save data to %q", s.datafile) } }