all: fix linting issues

Author: MariusVanDerWijden

.github/workflow/build-push.yaml

@@ -24,6 +24,9 @@ linters:
     - durationcheck
     - exportloopref
     - whitespace
+# this repo contains a few copied files from go-ethereum, 
+# and some of them have unused fields/functions
+    - unused 
 
     # - structcheck # lots of false positives
     # - errcheck #lot of false positives

.golangci.yml

@@ -87,7 +87,6 @@ func transferNewNodes(crawlerDB, nodeDB *sql.DB) error {
 		// they are normally recoverable, and a lot of the time, it's
 		// because the database is locked by the crawler.
 		return fmt.Errorf("error starting transaction to read nodes: %w", err)
-
 	}
 	defer crawlerDBTx.Rollback()
 

cmd/crawler/api.go

@@ -175,52 +175,49 @@ func (c *crawler) runIterator(done chan<- enode.Iterator, it enode.Iterator) {
 
 func (c *crawler) getClientInfoLoop() {
 	defer func() { c.Done() }()
-	for {
-		select {
-		case n, ok := <-c.reqCh:
-			if !ok {
-				return
-			}
+	for n := range c.reqCh {
+		if n == nil {
+			return
+		}
 
-			var tooManyPeers bool
-			var scoreInc int
+		var tooManyPeers bool
+		var scoreInc int
 
-			info, err := getClientInfo(c.genesis, c.networkID, c.nodeURL, n)
-			if err != nil {
-				log.Warn("GetClientInfo failed", "error", err, "nodeID", n.ID())
-				if strings.Contains(err.Error(), "too many peers") {
-					tooManyPeers = true
-				}
-			} else {
-				scoreInc = 10
+		info, err := getClientInfo(c.genesis, c.networkID, c.nodeURL, n)
+		if err != nil {
+			log.Warn("GetClientInfo failed", "error", err, "nodeID", n.ID())
+			if strings.Contains(err.Error(), "too many peers") {
+				tooManyPeers = true
 			}
+		} else {
+			scoreInc = 10
+		}
 
-			if info != nil {
-				log.Info(
-					"Updating node info",
-					"client_type", info.ClientType,
-					"version", info.SoftwareVersion,
-					"network_id", info.NetworkID,
-					"caps", info.Capabilities,
-					"fork_id", info.ForkID,
-					"height", info.Blockheight,
-					"td", info.TotalDifficulty,
-					"head", info.HeadHash,
-				)
-			}
+		if info != nil {
+			log.Info(
+				"Updating node info",
+				"client_type", info.ClientType,
+				"version", info.SoftwareVersion,
+				"network_id", info.NetworkID,
+				"caps", info.Capabilities,
+				"fork_id", info.ForkID,
+				"height", info.Blockheight,
+				"td", info.TotalDifficulty,
+				"head", info.HeadHash,
+			)
+		}
 
-			c.Lock()
-			node := c.output[n.ID()]
-			node.N = n
-			node.Seq = n.Seq()
-			if info != nil {
-				node.Info = info
-			}
-			node.TooManyPeers = tooManyPeers
-			node.Score += scoreInc
-			c.output[n.ID()] = node
-			c.Unlock()
+		c.Lock()
+		node := c.output[n.ID()]
+		node.N = n
+		node.Seq = n.Seq()
+		if info != nil {
+			node.Info = info
 		}
+		node.TooManyPeers = tooManyPeers
+		node.Score += scoreInc
+		c.output[n.ID()] = node
+		c.Unlock()
 	}
 }
 

pkg/crawler/crawl.go

@@ -6,7 +6,6 @@ import (
 )
 
 func TestParseVersionString(t *testing.T) {
-
 	type ParseTestCase struct {
 		name string
 		args string