treewide: replace error comparisons and assertions with errors.Is
Change-Id: Id2424eb155f2c6842c72c5fafd124d428ef901f2
Reviewed-on: https://review.monogon.dev/c/monogon/+/2994
Tested-by: Jenkins CI
Reviewed-by: Serge Bazanski <serge@monogon.tech>
diff --git a/metropolis/pkg/socksproxy/socksproxy.go b/metropolis/pkg/socksproxy/socksproxy.go
index dfd32c4..808ae1f 100644
--- a/metropolis/pkg/socksproxy/socksproxy.go
+++ b/metropolis/pkg/socksproxy/socksproxy.go
@@ -17,6 +17,7 @@
import (
"context"
+ "errors"
"fmt"
"io"
"log"
@@ -173,14 +174,14 @@
// Read request from the client and translate problems into early error replies.
req, err := readRequest(con)
- switch err {
- case errNotConnect:
+ switch {
+ case errors.Is(err, errNotConnect):
writeReply(con, ReplyCommandNotSupported, net.IPv4(0, 0, 0, 0), 0)
return
- case errUnsupportedAddressType:
+ case errors.Is(err, errUnsupportedAddressType):
writeReply(con, ReplyAddressTypeNotSupported, net.IPv4(0, 0, 0, 0), 0)
return
- case nil:
+ case err == nil:
default:
writeReply(con, ReplyGeneralFailure, net.IPv4(0, 0, 0, 0), 0)
return