swissmakers_gmbh/fail2ban-ui
12
1
Fork 0
mirror of https://github.com/swissmakers/fail2ban-ui.git synced 2026-04-17 05:53:15 +02:00
Code Issues Actions Packages Projects Releases Wiki Activity

Return better information on JSON parsing errors, that no action is recorded in that case

Browse Source
This commit is contained in:
Michael Reber
2026-02-20 15:30:53 +01:00
parent 9cdb2966ea
commit 5bef143fdb
1 changed files with 2 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
pkg/web/handlers.go
View File

@@ -292,7 +292,7 @@ func BanNotificationHandler(c *gin.Context) {
log.Printf("❌ Validation error: Field '%s' violated rule '%s'", fe.Field(), fe.ActualTag()) log.Printf("❌ Validation error: Field '%s' violated rule '%s'", fe.Field(), fe.ActualTag())
} }
} else { } else {
log.Printf("❌ JSON parsing error: %v", err) log.Printf("❌ JSON parsing error -> Action will not be recorded! Details: %v", err)
} }
log.Printf("Raw JSON: %s", string(body)) log.Printf("Raw JSON: %s", string(body))
c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid request: " + err.Error()}) c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid request: " + err.Error()})
@@ -361,7 +361,7 @@ func UnbanNotificationHandler(c *gin.Context) {
log.Printf("❌ Validation error: Field '%s' violated rule '%s'", fe.Field(), fe.ActualTag()) log.Printf("❌ Validation error: Field '%s' violated rule '%s'", fe.Field(), fe.ActualTag())
} }
} else { } else {
log.Printf("❌ JSON parsing error: %v", err) log.Printf("❌ JSON parsing error -> Action will not be recorded! Details: %v", err)
} }
log.Printf("Raw JSON: %s", string(body)) log.Printf("Raw JSON: %s", string(body))
c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid request: " + err.Error()}) c.JSON(http.StatusBadRequest, gin.H{"error": "Invalid request: " + err.Error()})
@@ -2448,7 +2448,6 @@ func getEmailStyle() string {
return "modern" return "modern"
} }
// Connects to the SMTP server and delivers a single HTML message. // Connects to the SMTP server and delivers a single HTML message.
func sendEmail(to, subject, body string, settings config.AppSettings) error { func sendEmail(to, subject, body string, settings config.AppSettings) error {
// Skips sending if the destination email is still the default placeholder // Skips sending if the destination email is still the default placeholder