Skip to content

Commit dca1156

Browse files
authored
chore: remove refs to deprecated io/ioutil (#459)
thanks @testwill
1 parent 4eb9e6a commit dca1156

File tree

9 files changed

+21
-26
lines changed

9 files changed

+21
-26
lines changed

client/client.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ import (
88
"encoding/base64"
99
"errors"
1010
"fmt"
11-
"io/ioutil"
1211
"net"
1312
"net/http"
1413
"net/url"
14+
"os"
1515
"regexp"
1616
"strings"
1717
"time"
@@ -118,7 +118,7 @@ func NewClient(c *Config) (*Client, error) {
118118
tc.InsecureSkipVerify = true
119119
} else if c.TLS.CA != "" {
120120
rootCAs := x509.NewCertPool()
121-
if b, err := ioutil.ReadFile(c.TLS.CA); err != nil {
121+
if b, err := os.ReadFile(c.TLS.CA); err != nil {
122122
return nil, fmt.Errorf("Failed to load file: %s", c.TLS.CA)
123123
} else if ok := rootCAs.AppendCertsFromPEM(b); !ok {
124124
return nil, fmt.Errorf("Failed to decode PEM: %s", c.TLS.CA)

main.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,6 @@ package main
33
import (
44
"flag"
55
"fmt"
6-
"io/ioutil"
76
"log"
87
"net/http"
98
"os"
@@ -89,7 +88,7 @@ var commonHelp = `
8988

9089
func generatePidFile() {
9190
pid := []byte(strconv.Itoa(os.Getpid()))
92-
if err := ioutil.WriteFile("chisel.pid", pid, 0644); err != nil {
91+
if err := os.WriteFile("chisel.pid", pid, 0644); err != nil {
9392
log.Fatal(err)
9493
}
9594
}

server/server_listen.go

+2-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import (
44
"crypto/tls"
55
"crypto/x509"
66
"errors"
7-
"io/ioutil"
87
"net"
98
"os"
109
"os/user"
@@ -116,7 +115,7 @@ func addCA(ca string, c *tls.Config) error {
116115
clientCAPool := x509.NewCertPool()
117116
if fileInfo.IsDir() {
118117
//this is a directory holding CA bundle files
119-
files, err := ioutil.ReadDir(ca)
118+
files, err := os.ReadDir(ca)
120119
if err != nil {
121120
return err
122121
}
@@ -140,7 +139,7 @@ func addCA(ca string, c *tls.Config) error {
140139
}
141140

142141
func addPEMFile(path string, pool *x509.CertPool) error {
143-
content, err := ioutil.ReadFile(path)
142+
content, err := os.ReadFile(path)
144143
if err != nil {
145144
return err
146145
}

share/cio/stdio.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,6 @@ package cio
22

33
import (
44
"io"
5-
"io/ioutil"
65
"os"
76
)
87

@@ -11,6 +10,6 @@ var Stdio = &struct {
1110
io.ReadCloser
1211
io.Writer
1312
}{
14-
ioutil.NopCloser(os.Stdin),
13+
io.NopCloser(os.Stdin),
1514
os.Stdout,
1615
}

share/settings/users.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import (
44
"encoding/json"
55
"errors"
66
"fmt"
7-
"io/ioutil"
7+
"os"
88
"regexp"
99
"sync"
1010

@@ -125,7 +125,7 @@ func (u *UserIndex) loadUserIndex() error {
125125
if u.configFile == "" {
126126
return errors.New("configuration file not set")
127127
}
128-
b, err := ioutil.ReadFile(u.configFile)
128+
b, err := os.ReadFile(u.configFile)
129129
if err != nil {
130130
return fmt.Errorf("Failed to read auth file: %s, error: %s", u.configFile, err)
131131
}

share/tunnel/tunnel.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ import (
44
"bytes"
55
"context"
66
"errors"
7-
"io/ioutil"
7+
"io"
88
"log"
99
"os"
1010
"sync"
@@ -57,7 +57,7 @@ func New(c Config) *Tunnel {
5757
//setup socks server (not listening on any port!)
5858
extra := ""
5959
if c.Socks {
60-
sl := log.New(ioutil.Discard, "", 0)
60+
sl := log.New(io.Discard, "", 0)
6161
if t.Logger.Debug {
6262
sl = log.New(os.Stdout, "[socks]", log.Ldate|log.Ltime)
6363
}

test/bench/main.go

+1-2
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@ import (
1717
"flag"
1818
"fmt"
1919
"io"
20-
"io/ioutil"
2120
"log"
2221
"net/http"
2322
"os"
@@ -86,7 +85,7 @@ func testTunnel(port string, size int) {
8685
fatal(err)
8786
}
8887

89-
n, err := io.Copy(ioutil.Discard, resp.Body)
88+
n, err := io.Copy(io.Discard, resp.Body)
9089
if err != nil {
9190
fatal(err)
9291
}

test/e2e/cert_utils_test.go

+7-8
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,6 @@ import (
1111
"crypto/x509/pkix"
1212
"encoding/pem"
1313
"fmt"
14-
"io/ioutil"
1514
"math/big"
1615
"net"
1716
"os"
@@ -53,7 +52,7 @@ func newTestTLSConfig() (*tlsConfig, error) {
5352
return nil, err
5453
}
5554

56-
tlsConfig.tmpDir, err = ioutil.TempDir("", "")
55+
tlsConfig.tmpDir, err = os.MkdirTemp("", "")
5756
if err != nil {
5857
return nil, err
5958
}
@@ -63,7 +62,7 @@ func newTestTLSConfig() (*tlsConfig, error) {
6362
return nil, err
6463
}
6564
pathServerCACrt := path.Join(dirServerCA, "client.crt")
66-
if err := ioutil.WriteFile(pathServerCACrt, clientCertPEM, 0666); err != nil {
65+
if err := os.WriteFile(pathServerCACrt, clientCertPEM, 0666); err != nil {
6766
return nil, err
6867
}
6968

@@ -72,7 +71,7 @@ func newTestTLSConfig() (*tlsConfig, error) {
7271
return nil, err
7372
}
7473
pathClientCACrt := path.Join(dirClientCA, "server.crt")
75-
if err := ioutil.WriteFile(pathClientCACrt, serverCertPEM, 0666); err != nil {
74+
if err := os.WriteFile(pathClientCACrt, serverCertPEM, 0666); err != nil {
7675
return nil, err
7776
}
7877

@@ -81,11 +80,11 @@ func newTestTLSConfig() (*tlsConfig, error) {
8180
return nil, err
8281
}
8382
pathServerCrtCrt := path.Join(dirServerCrt, "server.crt")
84-
if err := ioutil.WriteFile(pathServerCrtCrt, serverCertPEM, 0666); err != nil {
83+
if err := os.WriteFile(pathServerCrtCrt, serverCertPEM, 0666); err != nil {
8584
return nil, err
8685
}
8786
pathServerCrtKey := path.Join(dirServerCrt, "server.key")
88-
if err := ioutil.WriteFile(pathServerCrtKey, serverKeyPEM, 0666); err != nil {
87+
if err := os.WriteFile(pathServerCrtKey, serverKeyPEM, 0666); err != nil {
8988
return nil, err
9089
}
9190

@@ -94,11 +93,11 @@ func newTestTLSConfig() (*tlsConfig, error) {
9493
return nil, err
9594
}
9695
pathClientCrtCrt := path.Join(dirClientCrt, "client.crt")
97-
if err := ioutil.WriteFile(pathClientCrtCrt, clientCertPEM, 0666); err != nil {
96+
if err := os.WriteFile(pathClientCrtCrt, clientCertPEM, 0666); err != nil {
9897
return nil, err
9998
}
10099
pathClientCrtKey := path.Join(dirClientCrt, "client.key")
101-
if err := ioutil.WriteFile(pathClientCrtKey, clientKeyPEM, 0666); err != nil {
100+
if err := os.WriteFile(pathClientCrtKey, clientKeyPEM, 0666); err != nil {
102101
return nil, err
103102
}
104103

test/e2e/setup_test.go

+3-3
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@ package e2e_test
22

33
import (
44
"context"
5-
"io/ioutil"
5+
"io"
66
"log"
77
"net"
88
"net/http"
@@ -36,7 +36,7 @@ func (tl *testLayout) setup(t *testing.T) (server *chserver.Server, client *chcl
3636
fileAddr := "127.0.0.1:" + filePort
3737
f := http.Server{
3838
Handler: http.HandlerFunc(func(w http.ResponseWriter, r *http.Request) {
39-
b, _ := ioutil.ReadAll(r.Body)
39+
b, _ := io.ReadAll(r.Body)
4040
w.Write(append(b, '!'))
4141
}),
4242
}
@@ -133,7 +133,7 @@ func post(url, body string) (string, error) {
133133
if err != nil {
134134
return "", err
135135
}
136-
b, err := ioutil.ReadAll(resp.Body)
136+
b, err := io.ReadAll(resp.Body)
137137
if err != nil {
138138
return "", err
139139
}

0 commit comments

Comments
 (0)