Remove http from struct

Author: mtodor

internal/server/server.go

@@ -23,7 +23,6 @@ type Server struct {
 	cfg      *config.Config
 	registry *toolsets.Registry
 	mcp      *mcp.Server
-	http     *http.Server
 }
 
 // NewServer creates a new MCP server instance
@@ -89,7 +88,7 @@ func (s *Server) Start(ctx context.Context) error {
 	)
 
 	addr := fmt.Sprintf("%s:%d", s.cfg.Server.Address, s.cfg.Server.Port)
-	s.http = &http.Server{
+	httpServer := &http.Server{
 		Addr:    addr,
 		Handler: handler,
 	}
@@ -103,7 +102,7 @@ func (s *Server) Start(ctx context.Context) error {
 	// Start server in a goroutine
 	errChan := make(chan error, 1)
 	go func() {
-		if err := s.http.ListenAndServe(); err != nil && err != http.ErrServerClosed {
+		if err := httpServer.ListenAndServe(); err != nil && err != http.ErrServerClosed {
 			errChan <- fmt.Errorf("HTTP server error: %w", err)
 		}
 	}()
@@ -115,7 +114,7 @@ func (s *Server) Start(ctx context.Context) error {
 		// Create a context with timeout for graceful shutdown
 		shutdownCtx, shutdownCancel := context.WithTimeout(context.Background(), shutdownTimeout*time.Second)
 		defer shutdownCancel()
-		return errors.Wrap(s.http.Shutdown(shutdownCtx), "server shutting down failed")
+		return errors.Wrap(httpServer.Shutdown(shutdownCtx), "server shutting down failed")
 	case err := <-errChan:
 		return err
 	}