[BUG] Dynamic routes have higher priority over static routes

[Joshua-Tu]

[BUG] Dynamic routes have higher priority over static routes

[Joshua-Tu]

Hi, @alesancor1 noticed that there are still no replies on this one, is it going to be fixed in the next release?

[Joshua-Tu]

@alesancor1 Hi, could I know when this bug will be fixed?

[alesancor1]

It may take a while, there are many other things to be fixed and I'm currently working on other projects. I will come back to Oas Tools as soon as I can.

[alesancor1]

This issue has been fixed at eaf73a0. The problem was caused by the way middlewares were being registered, since each middleware was being registered for each path. I modified the code so every middleware is registered for a single path before moving to the next path for registration.

This way, the middleware chain now follows express logic. Static routes should be placed before dynamic routes in OAS Document in order for them to take precedence. The order they are registered in express is the same you write in the OAS Document.

[Joshua-Tu]

@alesancor1 is this fix in the incoming version 3.1.0 or the current latest version?

[alesancor1]

@Joshua-Tu I just published v3.1.0 containing this change :)