feat: add new error handler func with context parm

[pebo]

Add ErrorHandlerWithContext func to options. If both ErrorHandler and ErrorHandlerWithContext are set the later takes precedence.

Shows alternative fix for #11

[mikeschinkel]

@pebo — See any reason it should be limited to r.Context and not instead just pass in r?

I came here today because I need access to r.URL and r.Method so I can determine what content type I should return for the response.

Also, come to think of it, validateRequest(r *http.Request, router routers.Router, options *Options) should really also return the route that it matches, and then pass that to the error handler as well. Otherwise I'll have to maintain hard-coded information in my custom error handler that is already in openapi.yaml` file and should really only need to be here.

Would love to hear from the project owner if they will consider these or not.

[pebo]

@pebo — See any reason it should be limited to r.Context and not instead just pass in r?

I came here today because I need access to r.URL and r.Method so I can determine what content type I should return for the response.

Also, come to think of it, validateRequest(r *http.Request, router routers.Router, options *Options) should really also return the route that it matches, and then pass that to the error handler as well. Otherwise I'll have to maintain hard-coded information in my custom error handler that is already in openapi.yaml` file and should really only need to be here.

Would love to hear from the project owner if they will consider these or not.

I just needed the context and IIRC this is how the other middlewares worked. As for project owner, @jamietanna any chance this could be considered?

[mikeschinkel]

@pebo — Thank you for the quick reply.

I submitted an alternate PR (#15) where I tried to address your use-case, as well as address mine too. Hopefully it will meet your needs instead?

You can clone it from here and use a go.work file to see if it meets your needs.

-Mike