Skip to content

Commit ccb1b1b

Browse files
committed
log/rollbar: simplify getting the ENV & VERSION from env
Maybe env should support this itself.
1 parent 8de4965 commit ccb1b1b

1 file changed

Lines changed: 4 additions & 12 deletions

File tree

log/internal/rollbar/rollbar.go

Lines changed: 4 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -44,27 +44,19 @@ func Notify(err error) {
4444
}
4545

4646
func getEnvironment() string {
47-
pkgEnv := env.Get("ENV", env.Get("EQUINIX_ENV"))
48-
if pkgEnv == "" {
49-
// EQUINIX_ENV was not set! Checking for PACKET_ENV - Packet no longer exists, please switch
50-
pkgEnv = env.Get("ENV", env.Get("PACKET_ENV"))
51-
}
52-
47+
pkgEnv := env.Get("ENV", env.Get("EQUINIX_ENV", env.Get("PACKET_ENV")))
5348
if pkgEnv == "" {
5449
log.Panicw("required envvar(ENV) is unset", "envvar", "ENV")
5550
}
51+
5652
return pkgEnv
5753
}
5854

5955
func getVersion() string {
60-
version := env.Get("VERSION", env.Get("EQUINIX_VERSION"))
61-
if version == "" {
62-
// EQUINIX_VERSION was not set! Checking for PACKET_VERSION - Packet no longer exists, please switch
63-
version = env.Get("VERSION", env.Get("PACKET_VERSION"))
64-
}
65-
56+
version := env.Get("VERSION", env.Get("EQUINIX_VERSION", env.Get("PACKET_VERSION")))
6657
if version == "" {
6758
log.Panicw("required envvar(VERSION) is unset", "envvar", "VERSION")
6859
}
60+
6961
return version
7062
}

0 commit comments

Comments
 (0)