PullZoneHandler.onOriginRequest should accept Promise<Request | Response>

[nextend]

The current definition:

export declare type PullZoneHandler = {
    /**
     * Add a Middleware for the requests being processed.
     */
    onOriginRequest: (middleware: (ctx: OriginRequestContext) => Promise<Request> | Promise<Response>) => PullZoneHandler;
    /**
     * Add a Middleware for the response being processed.
     */
    onOriginResponse: (middleware: (ctx: OriginResponseContext) => Promise<Response>) => PullZoneHandler;
};

But I think it should also accept Promise<Request | Response> return type as the response might depend on the body of the callback function. Imagine a middleware, which return a Response on authentication failure, but returns the original Request on success.

Suggested:

export declare type PullZoneHandler = {
    /**
     * Add a Middleware for the requests being processed.
     */
    onOriginRequest: (middleware: (ctx: OriginRequestContext) => Promise<Request> | Promise<Response> | Promise<Request | Response>) => PullZoneHandler;
    /**
     * Add a Middleware for the response being processed.
     */
    onOriginResponse: (middleware: (ctx: OriginResponseContext) => Promise<Response>) => PullZoneHandler;
};

[antho-bunny]

In fact it could just return a Promise<Request | Response> and instead of the Promise<Request> | Promise<Response> | Promise<Request | Response>

If you change your request to this & include a little changeset over it it'll be merged in the following days :-).

Thank you for the PR 🙏

[nextend]

done