db123/auth-proxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: use sql.NullString for apiClientIP in logs query

Browse Source 
Change apiClientIP from string to sql.NullString to properly handle
NULL values from the database, preventing potential runtime errors
when the column contains NULL entries.
This commit is contained in:
db123-test
2026-05-07 17:46:23 +03:30
parent 6ea4019d55
commit 3ba6d61117
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/auth.go
View File

@@ -302,8 +302,8 @@ func logsHandler(db *sql.DB) http.HandlerFunc {
logs := make([]map[string]interface{}, 0) logs := make([]map[string]interface{}, 0)
for rows.Next() { for rows.Next() {
var ts time.Time var ts time.Time
var action, apiClientIP string var action string
var ip, reason sql.NullString var ip, reason, apiClientIP sql.NullString
var ttlSeconds sql.NullInt64 var ttlSeconds sql.NullInt64
if err := rows.Scan(&ts, &action, &ip, &ttlSeconds, &reason, &apiClientIP); err != nil { if err := rows.Scan(&ts, &action, &ip, &ttlSeconds, &reason, &apiClientIP); err != nil {
continue continue