popmd: remove unused 'handle' func and return err in connectBFG (#124)

service/popm/popm.go

@@ -12,7 +12,6 @@ import (
 	"encoding/hex"
 	"errors"
 	"fmt"
-	"net/http"
 	"slices"
 	"strings"
 	"sync"
@@ -663,11 +662,6 @@ func (m *Miner) promRunning() float64 {
 	return 0
 }
 
-func handle(service string, mux *http.ServeMux, pattern string, handler func(http.ResponseWriter, *http.Request)) {
-	mux.HandleFunc(pattern, handler)
-	log.Infof("handle (%v): %v", service, pattern)
-}
-
 func (m *Miner) handleBFGCallCompletion(parrentCtx context.Context, conn *protocol.Conn, bc bfgCmd) {
 	log.Tracef("handleBFGCallCompletion")
 	defer log.Tracef("handleBFGCallCompletion exit")
@@ -805,7 +799,7 @@ func (m *Miner) connectBFG(pctx context.Context) error {
 	log.Debugf("Connected to BFG: %s", m.cfg.BFGWSURL)
 	m.bfgWg.Wait()
 
-	return nil
+	return err
 }
 
 func (m *Miner) bfg(ctx context.Context) error {