Merge pull request #23 from Sagit-chu/opencode/crisp-cabin

fix(gost): remove single-node optimization to enable forwarder failover
2026-11-14 16:54:24 +08:00 · 2026-02-05 10:18:00 +08:00 · 2026-02-05 02:14:37 +00:00
1 changed files with 3 additions and 9 deletions
@@ -204,17 +204,11 @@ func (p *chainHop) Select(ctx context.Context, opts ...hop.SelectOption) *chain.
 		return nodes[0]
 	}

-	// For single-node case: bypass selector/FailFilter to ensure availability.
-	// The marker system still works for metrics, but we don't block requests
-	// based on recent failures - the connection will be attempted regardless.
-	// This matches upstream go-gost/x behavior.
-	if len(nodes) == 1 {
-		return nodes[0]
-	}
-
-	// Multi-node case: use selector with FailFilter for proper failover.
+	// Use selector with FailFilter for proper failover.
 	// FailFilter will exclude recently-failed nodes, allowing traffic to
 	// be routed to healthy alternatives.
+	// Note: FailFilter has a safety guard (len <= 1 returns as-is) to ensure
+	// the last remaining node is never permanently blocked.
 	if s := p.options.selector; s != nil {
 		log.Debugf("[hop.Select] calling selector.Select with %d nodes", len(nodes))
 		if node := s.Select(ctx, nodes...); node != nil {