Wissance / Ferrum

Simple Go OpenId authorization server with Keycloak compatible API
https://wissance.github.io/Ferrum/
Apache License 2.0
5 stars 4 forks source link

Logger could crush app if config is missing or invalid #45

Closed EvilLord666 closed 6 months ago

EvilLord666 commented 11 months ago

we should check and fix

awsoremod commented 11 months ago

Внутри этой функции:

application/application.go
func (app *Application) Init() (bool, error) 

Логгер используется внутри функции в функциях  (внутри этих функций логгер nil):
app.readAppConfig()
app.readKey()

Но логгер создается после использования этих функций.

EvilLord666 commented 11 months ago

Sure it seems that error is logging to logger but app is not configured yet, probably we should output just with fmt.Println(msg), until app.logger.Init() is called

EvilLord666 commented 6 months ago

Done