Code maintenance: remove unneeded defaultInterface in Shadowsocks

cmd/gluetun/main.go

@@ -265,7 +265,7 @@ func _main(background context.Context, buildInfo models.BuildInformation,
 	wg.Add(1)
 	go httpProxyLooper.Run(ctx, wg)
 
-	shadowsocksLooper := shadowsocks.NewLooper(allSettings.ShadowSocks, logger, defaultInterface)
+	shadowsocksLooper := shadowsocks.NewLooper(allSettings.ShadowSocks, logger)
 	restartShadowsocks := shadowsocksLooper.Restart
 	wg.Add(1)
 	go shadowsocksLooper.Run(ctx, wg)

internal/shadowsocks/loop.go

@@ -21,14 +21,13 @@ type Looper interface {
 }
 
 type looper struct {
-	settings         settings.ShadowSocks
+	settings      settings.ShadowSocks
-	settingsMutex    sync.RWMutex
+	settingsMutex sync.RWMutex
-	logger           logging.Logger
+	logger        logging.Logger
-	defaultInterface string
+	restart       chan struct{}
-	restart          chan struct{}
+	start         chan struct{}
-	start            chan struct{}
+	stop          chan struct{}
-	stop             chan struct{}
+	backoffTime   time.Duration
-	backoffTime      time.Duration
 }
 
 func (l *looper) logAndWait(ctx context.Context, err error) {
@@ -47,15 +46,14 @@ func (l *looper) logAndWait(ctx context.Context, err error) {
 
 const defaultBackoffTime = 10 * time.Second
 
-func NewLooper(settings settings.ShadowSocks, logger logging.Logger, defaultInterface string) Looper {
+func NewLooper(settings settings.ShadowSocks, logger logging.Logger) Looper {
 	return &looper{
-		settings:         settings,
+		settings:    settings,
-		logger:           logger.WithPrefix("shadowsocks: "),
+		logger:      logger.WithPrefix("shadowsocks: "),
-		defaultInterface: defaultInterface,
+		restart:     make(chan struct{}),
-		restart:          make(chan struct{}),
+		start:       make(chan struct{}),
-		start:            make(chan struct{}),
+		stop:        make(chan struct{}),
-		stop:             make(chan struct{}),
+		backoffTime: defaultBackoffTime,
-		backoffTime:      defaultBackoffTime,
 	}
 }