chore:added a logrus hook for sentry

docker-compose.yaml

@@ -5,10 +5,13 @@ services:
     restart: always
     build:
       context: server/
+      args:
+        version: dev # compiled with the git sha in prod
     ports:
       - 50051:50051
     environment:
       - DB_DSN=root:${DB_ROOT_PASSWORD}@tcp(db:${DB_PORT:-3306})/${DB_NAME}?charset=utf8mb4&parseTime=True&loc=Local
+      - ENVIRONMENT=dev
       - SENTRY_DSN=${SENTRY_DSN}
       - APNS_KEY_ID=${APNS_KEY_ID}
       - APNS_TEAM_ID=${APNS_TEAM_ID}

server/backend/cron/fileDownload.go

@@ -6,7 +6,6 @@ import (
 	"github.com/TUM-Dev/Campus-Backend/server/model"
 	"github.com/disintegration/imaging"
 	"github.com/gabriel-vasile/mimetype"
-	"github.com/getsentry/sentry-go"
 	log "github.com/sirupsen/logrus"
 	"gorm.io/gorm"
 	"image"
@@ -43,7 +42,6 @@ func (c *CronService) downloadFile(file model.Files) {
 	resp, err := http.Get(url)
 	if err != nil {
 		log.WithError(err).WithField("url", url).Warn("Could not download image")
-		sentry.CaptureException(err)
 		return
 	}
 	// read body here because we can't exhaust the io.reader twice
@@ -59,7 +57,6 @@ func (c *CronService) downloadFile(file model.Files) {
 		downloadedImg, _, err := image.Decode(bytes.NewReader(body))
 		if err != nil {
 			log.WithError(err).WithField("url", url).Warn("Couldn't decode source image")
-			sentry.CaptureException(err)
 			return
 		}
 
@@ -69,21 +66,18 @@ func (c *CronService) downloadFile(file model.Files) {
 		err = imaging.Save(dstImage, StorageDir+dstFileName, imaging.JPEGQuality(75))
 		if err != nil {
 			log.WithError(err).WithField("url", url).Warn("Could not save image file")
-			sentry.CaptureException(err)
 			return
 		}
 	} else {
 		// save without resizing image
 		err = os.WriteFile(fmt.Sprintf("%s%s", file.Path, file.Name), body, 0644)
 		if err != nil {
-			sentry.CaptureException(err)
 			log.WithError(err).Error("Can't save file to disk")
 			return
 		}
 	}
 	err = c.db.Model(&model.Files{}).Where("url = ?", url).Update("downloaded", true).Error
 	if err != nil {
-		sentry.CaptureException(err)
 		log.WithError(err).Error("Could not set image to downloaded.")
 	}
 }

server/backend/cron/news.go

@@ -6,7 +6,6 @@ import (
 	"errors"
 	"fmt"
 	"github.com/TUM-Dev/Campus-Backend/server/model"
-	"github.com/getsentry/sentry-go"
 	"github.com/guregu/null"
 	"github.com/microcosm-cc/bluemonday"
 	"github.com/mmcdole/gofeed"
@@ -44,8 +43,7 @@ func (c *CronService) newsCron(cronjob *model.Crontab) error {
 	var source model.NewsSource
 	err := c.db.Find(&source, cronjob.ID.Int64).Error
 	if err != nil {
-		log.Printf("error getting news source from database: %v", err)
-		sentry.CaptureException(err)
+		log.WithError(err).Error("getting news source from database")
 		return err
 	}
 	// skip sources with null url
@@ -68,15 +66,13 @@ func (c *CronService) parseNewsFeed(source model.NewsSource) error {
 	log.WithField("url", source.URL.String).Trace("processing newsfeed")
 	feed, err := c.gf.ParseURL(source.URL.String)
 	if err != nil {
-		log.Printf("error parsing rss: %v", err)
-		sentry.CaptureException(err)
+		log.WithError(err).Error("parsing rss")
 		return err
 	}
 	// get all news for this source so we only process new ones, using map for performance reasons
 	existingNewsLinksForSource := make([]string, 0)
 	if err := c.db.Table("`news`").Select("`link`").Where("`src` = ?", source.Source).Scan(&existingNewsLinksForSource).Error; err != nil && !errors.Is(err, gorm.ErrRecordNotFound) {
-		log.Printf("failed to fetch existing news: %v", err)
-		sentry.CaptureException(err)
+		log.WithError(err).Error("failed to fetch existing news")
 		return err
 	}
 	var newNews []model.News
@@ -108,8 +104,7 @@ func (c *CronService) parseNewsFeed(source model.NewsSource) error {
 			if pickedEnclosure != nil {
 				fileId, err = c.saveImage(pickedEnclosure.URL)
 				if err != nil {
-					log.WithError(err).Error("error saving image")
-					sentry.CaptureException(err)
+					log.WithError(err).Error("can't save news image")
 				}
 				enclosureUrl = null.String{NullString: sql.NullString{String: pickedEnclosure.URL, Valid: true}}
 			}
@@ -195,7 +190,6 @@ func (c *CronService) cleanOldNewsForSource(source int32) error {
 		log.WithField("RowsAffected", res.RowsAffected).Info("cleaned up old news")
 	} else {
 		log.WithError(res.Error).Error("failed to clean up old news")
-		sentry.CaptureException(res.Error)
 		return res.Error
 	}
 	return nil

server/go.mod

@@ -11,6 +11,7 @@ require (
 	github.com/grpc-ecosystem/grpc-gateway/v2 v2.17.1
 	github.com/guregu/null v4.0.0+incompatible
 	github.com/influxdata/influxdb-client-go/v2 v2.12.3
+	github.com/makasim/sentryhook v0.4.2
 	github.com/microcosm-cc/bluemonday v1.0.25
 	github.com/mmcdole/gofeed v1.2.1
 	github.com/satori/go.uuid v1.2.0
@@ -41,7 +42,6 @@ require (
 	github.com/jinzhu/inflection v1.0.0 // indirect
 	github.com/jinzhu/now v1.1.5 // indirect
 	github.com/json-iterator/go v1.1.12 // indirect
-	github.com/kr/text v0.2.0 // indirect
 	github.com/mmcdole/goxpp v1.1.0 // indirect
 	github.com/modern-go/concurrent v0.0.0-20180306012644-bacd9c7ef1dd // indirect
 	github.com/modern-go/reflect2 v1.0.2 // indirect