flamingo
flamingo copied to clipboard
web: Add context to error logs
Hello friends,
while analysing an issue with our SSO by investigating our application logs, I noticed that the function called here: https://github.com/i-love-flamingo/flamingo/blob/master/core/auth/oauth/oidc.go#L310
Does not consider the request context when logging the error https://github.com/i-love-flamingo/flamingo/blob/master/framework/web/result.go#L430
This would require the function to be able to receive a context
and log the error using WithContext
The adjustment would help with log analysis.
As this function is used across several classes, we could also create a new function and slowly migrate.