style: lint

cmd/regioncheck/linode.go

@@ -28,7 +28,7 @@ func GetRegionsLinode() ([]string, error) {
 	regions := []string{}
 	doc.Find(".rdmd-table:nth-of-type(1) tbody tr td:nth-of-type(4)").Each(func(_ int, t *goquery.Selection) {
 		for _, r := range regionRe.FindAllString(t.Text(), -1) {
-			regions = append(regions, strings.Replace(r, ".linodeobjects.com", "", -1))
+			regions = append(regions, strings.ReplaceAll(r, ".linodeobjects.com", ""))
 		}
 	})
 

provider/aws.go

@@ -4,11 +4,12 @@ import (
 	"context"
 	"errors"
 	"fmt"
+	"net/http"
+	"time"
+
 	"github.com/aws/aws-sdk-go-v2/aws"
 	"github.com/aws/aws-sdk-go-v2/feature/ec2/imds"
 	"github.com/sa7mon/s3scanner/permission"
-	"net/http"
-	"time"
 
 	awshttp "github.com/aws/aws-sdk-go-v2/aws/transport/http"
 	"github.com/aws/aws-sdk-go-v2/config"
@@ -73,10 +74,7 @@ func (a *AWS) Scan(b *bucket.Bucket, doDestructiveChecks bool) error {
 }
 
 func (a *AWS) Enumerate(b *bucket.Bucket) error {
-	useCreds := false
-	if b.PermAuthUsersRead == bucket.PermissionAllowed {
-		useCreds = true
-	}
+	useCreds := b.PermAuthUsersRead == bucket.PermissionAllowed
 	client, err := a.getRegionClient(b.Region, useCreds)
 	if err != nil {
 		return err

provider/clientmap/clientmap.go

@@ -1,8 +1,9 @@
 package clientmap
 
 import (
-	"github.com/aws/aws-sdk-go-v2/service/s3"
 	"sync"
+
+	"github.com/aws/aws-sdk-go-v2/service/s3"
 )
 
 type ClientKey struct {
@@ -22,10 +23,10 @@ func New() *ClientMap {
 	}
 }
 
-func WithCapacity(cap int) *ClientMap {
+func WithCapacity(capacity int) *ClientMap {
 	return &ClientMap{
 		Mutex: sync.Mutex{},
-		inner: make(map[ClientKey]*s3.Client, cap),
+		inner: make(map[ClientKey]*s3.Client, capacity),
 	}
 }
 

provider/credentials.go

@@ -3,6 +3,7 @@ package provider
 import (
 	"context"
 	"errors"
+
 	"github.com/aws/aws-sdk-go-v2/aws"
 	"github.com/aws/aws-sdk-go-v2/service/s3"
 	"github.com/aws/smithy-go"
@@ -18,7 +19,7 @@ func HasCredentials(cfg aws.Config) (bool, string) {
 	if credsErr != nil {
 		var oe *smithy.OperationError
 		if errors.As(credsErr, &oe) {
-			if !(oe.ServiceID == "ec2imds" && oe.OperationName == "GetMetadata") {
+			if oe.ServiceID != "ec2imds" || oe.OperationName != "GetMetadata" {
 				log.WithFields(log.Fields{"method": "provider.HasCredentials"}).Error(oe.Error())
 			}
 			return false, ""
@@ -32,7 +33,7 @@ func ClientHasCredentials(client *s3.Client) bool {
 	if credsErr != nil {
 		var oe *smithy.OperationError
 		if errors.As(credsErr, &oe) {
-			if !(oe.ServiceID == "ec2imds" && oe.OperationName == "GetMetadata") {
+			if oe.ServiceID != "ec2imds" || oe.OperationName != "GetMetadata" {
 				log.WithFields(log.Fields{"method": "provider.ClientHasCredentials"}).Error(oe.Error())
 			}
 			return false

provider/custom.go

@@ -80,11 +80,12 @@ func NewCustomProvider(addressStyle string, insecure bool, regions []string, end
 	cp.regions = regions
 	cp.insecure = insecure
 	cp.endpointFormat = endpointFormat
-	if addressStyle == "path" {
+	switch addressStyle {
+	case "path":
 		cp.addressStyle = PathStyle
-	} else if addressStyle == "vhost" {
+	case "vhost":
 		cp.addressStyle = VirtualHostStyle
-	} else {
+	default:
 		return cp, fmt.Errorf("unknown custom provider address style: %s. Expected 'path' or 'vhost'", addressStyle)
 	}
 
@@ -99,7 +100,7 @@ func NewCustomProvider(addressStyle string, insecure bool, regions []string, end
 func (cp *CustomProvider) newClients() (*clientmap.ClientMap, error) {
 	clients := clientmap.WithCapacity(len(cp.regions))
 	for _, r := range cp.regions {
-		regionURL := strings.Replace(cp.endpointFormat, "$REGION", r, -1)
+		regionURL := strings.ReplaceAll(cp.endpointFormat, "$REGION", r)
 		client, err := newNonAWSClient(cp, regionURL)
 		if err != nil {
 			return nil, err