[FEATURE REQUEST] Handler function middleware about asynq HOT 3 CLOSED

For most use cases this works fine but in cases where you have a lot of middleware functions having something like:
mux.Use(someMiddlware) or mux.Add(someMiddleware) it would help avoid the clutter.

from asynq.

@rugwirobaker Thanks for this feature request!

Could you provide an example code for a potential API for middleware support?

Alternatively, can you simply write a middleware function and wrap your handlers with that middleware?

func loggingMiddleware(h asynq.Handler) asynq.Handler {
    return asynq.HandlerFunc(func(ctx context.Context, t *asynq.Task) error {
        // log something before handler processes the task
        h.ProcessTask(ctx, t)
        // log something after handler processed the task
    })
}

mux := asynq.ServeMux()
mux.HandleFunc("something", loggingMiddleware(myHandler))

would that work? Or should the package offer better support for middleware chain?

from asynq.

Ok that makes sense.
Let's add Use method to add middlewares to the chain, similar to gorilla/mux#Route.Use. The method should be a variadic function that takes any number of middleware functions. They should be executed in the order that they are applied.

Example:

mux := asynq.ServeMux()
mux.Use(loggingMiddleware)
mux.HandleFunc("something", myHandler)

from asynq.