From 27f902041b9eec9b9eab57f11155e7a6fda24740 Mon Sep 17 00:00:00 2001
From: Ludvig Lundgren <hello@ludviglundgren.se>
Date: Mon, 10 Jan 2022 21:18:11 +0100
Subject: [PATCH] fix: make valid channels check lowercase (#75)

---
 internal/irc/handler.go | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/internal/irc/handler.go b/internal/irc/handler.go
index 140b0bb..c6c5338 100644
--- a/internal/irc/handler.go
+++ b/internal/irc/handler.go
@@ -85,6 +85,7 @@ func NewHandler(network domain.IrcNetwork, filterService filter.Service, release
 	for _, definition := range definitions {
 		// indexers can use multiple channels, but it's not common, but let's handle that anyway.
 		for _, channel := range definition.IRC.Channels {
+			// some channels are defined in mixed case
 			channel = strings.ToLower(channel)
 
 			h.announceProcessors[channel] = announce.NewAnnounceProcessor(definition, filterService, releaseService)
@@ -93,17 +94,15 @@ func NewHandler(network domain.IrcNetwork, filterService filter.Service, release
 				name:       channel,
 				monitoring: false,
 			}
+
+			// create map of valid channels
+			h.validChannels[channel] = struct{}{}
 		}
 
 		// create map of valid announcers
 		for _, announcer := range definition.IRC.Announcers {
 			h.validAnnouncers[announcer] = struct{}{}
 		}
-
-		// create map of valid channels
-		for _, ch := range definition.IRC.Channels {
-			h.validChannels[ch] = struct{}{}
-		}
 	}
 
 	return h
@@ -622,7 +621,7 @@ func (s *Handler) isValidAnnouncer(nick string) bool {
 
 // check if channel is one from the list in the definition
 func (s *Handler) isValidChannel(channel string) bool {
-	_, ok := s.validChannels[channel]
+	_, ok := s.validChannels[strings.ToLower(channel)]
 	if !ok {
 		return false
 	}