Handle error for new telegram client

Author: richardgreg

internal/alert/routing.go

@@ -3,8 +3,11 @@
 package alert
 
 import (
+	"go.uber.org/zap"
+
 	"github.com/base-org/pessimism/internal/client"
 	"github.com/base-org/pessimism/internal/core"
+	"github.com/base-org/pessimism/internal/logging"
 )
 
 // RoutingDirectory ... Interface for routing directory
@@ -112,7 +115,11 @@ func (rd *routingDirectory) paramsToRouteDirectory(acc *core.AlertClientCfg, sev
 				ChatID: cfg.ChatID.String(),
 				Token:  cfg.Token.String(),
 			}
-			client := client.NewTelegramClient(conf, name)
+			client, err := client.NewTelegramClient(conf, name)
+			if err != nil {
+				logging.NoContext().Error("Failed to create Telegram client", zap.String("name", name), zap.Error(err))
+				continue
+			}
 			rd.telegramClients[sev] = append(rd.telegramClients[sev], client)
 		}
 	}

internal/client/telegram.go

@@ -4,6 +4,7 @@ import (
 	"bytes"
 	"context"
 	"encoding/json"
+	"errors"
 	"fmt"
 	"io"
 	"net/http"
@@ -28,17 +29,18 @@ type telegramClient struct {
 	client *http.Client
 }
 
-func NewTelegramClient(cfg *TelegramConfig, name string) TelegramClient {
+func NewTelegramClient(cfg *TelegramConfig, name string) (TelegramClient, error) {
 	if cfg.Token == "" {
-		logging.NoContext().Warn("No Telegram token provided")
+		logging.NoContext().Warn("No Telegram bot token provided")
+		return nil, errors.New("No Telegram bot token was provided")
 	}
 
 	return &telegramClient{
 		token:  cfg.Token,
 		chatID: cfg.ChatID,
 		name:   name,
 		client: &http.Client{},
-	}
+	}, nil
 }
 
 type TelegramPayload struct {