fix: geoip lookup should be try-caught

Signed-off-by: Arnav Gupta <championswimmer@gmail.com>
championswimmer · Feb 24, 2024 · dddbba9 · dddbba9
1 parent 106fd92
commit dddbba9
Showing 1 changed file with 6 additions and 5 deletions.
diff --git a/src/controllers/events.go b/src/controllers/events.go
@@ -71,12 +71,13 @@ func (c *EventsController) LogRedirectAsync(redirData *EventRedirectData) {
 			IPAddress:  redirData.IPAddress,
 			Referer:    redirData.Referer,
 		}
-		geoIpData, err := clientinfo.GetGeoIpDataFromIP(c.geoipDB, redirData.IPAddress)
-		if err != nil {
-			applogger.Warn("LogRedirectAsync: failed to get geoip data: ", err)
-		} else {
+		lo.TryCatchWithErrorValue(func() error {
+			geoIpData, err := clientinfo.GetGeoIpDataFromIP(c.geoipDB, redirData.IPAddress)
 			event.GeoIpData = *geoIpData
-		}
+			return err
+		}, func(e any) {
+			applogger.Warn("LogRedirectAsync: failed to get geoip data: ", e)
+		})
 
 		lo.Try(func() error {
 			tx := c.eventDb.Create(event)