denoland/deno_blog

The order of middleware is changed round and round every time.

s-uei opened this issue · 1 comments

s-uei commented

Hi.

I found a bug in composeMiddlewares.

deno_blog/blog.tsx

Line 138 in 36cc6c3

const mws = state.middlewares?.reverse();

The reverse function change the array of caller by breaking. The order of state.middlrewares change every time.

I recommend using slice().reverse() instead of reverse().

lino-levan commented

#93 got merged, shouldn't this issue be closed?

  • 👍1