rledisez

Results 9 comments of rledisez

Try to be more user friendly on error messages Any feedbacks appreciated @antihax @grokify @kemokemo @jirikuncar @ph4r5h4d @lwj5

Actually, I changed a bit the way to do it to add a Param field to ErrParsing so it can apply to all parsing error (eg: WithRequire, WithParse, ...). So...

By applying the default values, this make the usage of default values of a post body consistent with the default values of query parameters Any feedbacks appreciated @antihax @grokify @kemokemo...

> Does the default work when there are nested objects? > > such as embedded struct, and when a struct is a type for one of the fields. The previous...

The case it avoids is this one: ``` goroutine 86515 [chan send, 7 minutes]: github.com/Jille/raft-grpc-transport.gRPCAPI.handleRPC({0xc07a412180?, {0x0?, 0x0?}}, {0xf5cd80?, 0xc1caac0120}, {0x0, 0x0}) /home/administrator/go/pkg/mod/github.com/!jille/[email protected]/grpcapi.go:37 +0x1ec github.com/Jille/raft-grpc-transport.gRPCAPI.AppendEntriesPipeline({0xc07a412180?, {0x0?, 0x0?}}, {0x12093b0, 0xc07a3e0c30}) /home/administrator/go/pkg/mod/github.com/!jille/[email protected]/grpcapi.go:111 +0xb2...

@Jille is there anything I can do to help unblock this PR?

Thx for your feedback. I updated the template to set default values for embedded structs. I created a test as `samples/openapi3/server/petstore/go/model_store_test.go` but I'm not sure this will be executed as...

How can we unblock (this much needed) feature?

Anything we can do to have this merged?