Skip to content

Commit 00097eb

Browse files
committed
fix(receive): improve error messages in form to better reflect the issue
1 parent 5eff140 commit 00097eb

1 file changed

Lines changed: 13 additions & 3 deletions

File tree

  • internal/project/lib/core/receive

internal/project/lib/core/receive/form.go

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,10 @@ var FormMetadataCacheMutex sync.Mutex
1919
// client and stores it in the value pointed to by value.
2020
func Form(client *clients.Client, value any) bool {
2121
if client.WebSocket != nil {
22-
client.Options.ErrorLog.Println("web socket connections cannot parse forms", stack.Trace())
22+
client.Options.ErrorLog.Printf(
23+
"receive.Form: web socket connections cannot parse forms\n%s",
24+
stack.Trace(),
25+
)
2326
return false
2427
}
2528
isMultipart := true
@@ -28,14 +31,21 @@ func Form(client *clients.Client, value any) bool {
2831
if errors.Is(err, http.ErrNotMultipart) {
2932
isMultipart = false
3033
} else {
31-
client.Options.ErrorLog.Println(err, stack.Trace())
34+
client.Options.ErrorLog.Printf(
35+
"receive.Form: failed to parse multipart form: %v\n%s",
36+
err,
37+
stack.Trace(),
38+
)
3239
return false
3340
}
3441
}
3542
}
3643
reflection := reflect.ValueOf(value)
3744
if reflection.Kind() != reflect.Pointer {
38-
client.Options.ErrorLog.Println("form value must be a pointer", stack.Trace())
45+
client.Options.ErrorLog.Printf(
46+
"receive.Form: form value must be a pointer\n%s",
47+
stack.Trace(),
48+
)
3949
return false
4050
}
4151
reflection = reflection.Elem()

0 commit comments

Comments
 (0)