Merge pull request #49 from brenkem/fix-string-conversion

kha7iq · web-flow · commit 24242b5da2da · 2021-06-21T23:10:32.000+08:00
fix: use strconv.ParseInt instead of strconv.Atoi
diff --git a/service/telegram/telegram.go b/service/telegram/telegram.go
@@ -78,11 +78,11 @@ All configuration options are also available via environment variables.`,
 				if len(v) <= 0 {
 					return helpers.ErrChannel
 				}
-				k, errStr := strconv.Atoi(v)
+				k, errStr := strconv.ParseInt(v, 10, 64)
 				if errStr != nil {
 					return errStr
 				}
-				telegramSvc.AddReceivers(int64(k))
+				telegramSvc.AddReceivers(k)
 			}
 
 			notifier.UseServices(telegramSvc)

Original file line number	Diff line number	Diff line change
@@ -78,11 +78,11 @@ All configuration options are also available via environment variables.`,
`78`	`78`	`if len(v) <= 0 {`
`79`	`79`	`return helpers.ErrChannel`
`80`	`80`	`}`
`81`		`- k, errStr := strconv.Atoi(v)`
	`81`	`+ k, errStr := strconv.ParseInt(v, 10, 64)`
`82`	`82`	`if errStr != nil {`
`83`	`83`	`return errStr`
`84`	`84`	`}`
`85`		`- telegramSvc.AddReceivers(int64(k))`
	`85`	`+ telegramSvc.AddReceivers(k)`
`86`	`86`	`}`
`87`	`87`
`88`	`88`	`notifier.UseServices(telegramSvc)`