-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
log client_ip field with real ip value (#209) #210
Conversation
i think it would be better to log both, +1 for pull request+tests |
@p53 changed to log both, thanks |
r.log.Debug("upgrading the connnection", zap.String("client_ip", req.RemoteAddr)) | ||
clientIP := utils.RealIP(req) | ||
r.log.Debug("upgrading the connnection", | ||
zap.String("client_ip", clientIP), |
Check failure
Code scanning / CodeQL
Log entries created from user input
|
||
scope.Logger.Debug( | ||
"incoming authorization request from client address", | ||
zap.Any("access_type", accessType), | ||
zap.String("auth_url", authURL), | ||
zap.String("client_ip", req.RemoteAddr), | ||
zap.String("client_ip", clientIP), |
Check failure
Code scanning / CodeQL
Log entries created from user input
scope.Logger.Error(errorMsg, | ||
zap.String("client_ip", req.RemoteAddr), | ||
zap.String("client_ip", clientIP), |
Check failure
Code scanning / CodeQL
Log entries created from user input
log client_ip field with real ip value (#209)
Summary
Type
[] Bug fix
[] Feature request
[+] Enhancement
[] Docs
Why?
Requirements
How to try it?
Documentation
Additional Information
Checklist: