Skip to content

Commit

Permalink
Add better errors to providers init
Browse files Browse the repository at this point in the history
  • Loading branch information
KowalskiPiotr98 committed Dec 31, 2024
1 parent 22c7278 commit 6d62aae
Show file tree
Hide file tree
Showing 3 changed files with 14 additions and 4 deletions.
4 changes: 3 additions & 1 deletion auth/errors.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,5 +3,7 @@ package auth
import "errors"

var (
UserIdNotStored = errors.New("user id was not stored in user session")
UserIdNotStored = errors.New("user id was not stored in user session")
NoProvidersSet = errors.New("no providers were set during initialisation")
ProvidersAlreadyInitialised = errors.New("providers were already initialised")
)
10 changes: 8 additions & 2 deletions auth/providers.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,10 +28,10 @@ var (
}
)

func SetupProviders(baseUrl string) {
func SetupProviders(baseUrl string) error {
if areProvidersSet() {
// prevent duplicate provider setup
return
return ProvidersAlreadyInitialised
}

baseUrl = strings.TrimRight(baseUrl, "/")
Expand All @@ -50,6 +50,12 @@ func SetupProviders(baseUrl string) {
}

goth.UseProviders(enabledProviders...)

if !areProvidersSet() {
return NoProvidersSet
}

return nil
}

func GetEnabledProviders() []string {
Expand Down
4 changes: 3 additions & 1 deletion main.go
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,9 @@ func runEngine() error {
controllers.SetRoutes(router.Group("/api/v1"))

auth.InitSessionStore(listenDomain)
auth.SetupProviders(baseAddress.String())
if err = auth.SetupProviders(baseAddress.String()); err != nil {
log.Panicf("Failed to setup login providers: %v", err)
}

log.Infoln("Starting server...")
return router.Run(listenAddress)
Expand Down

0 comments on commit 6d62aae

Please sign in to comment.