search

stackvana / hook.io

Open-Source Microservice Hosting Platform
https://hook.io
Other
1.26k stars 117 forks source link

Duplicate /logout handler #195

Closed pyhedgehog closed 8 years ago

pyhedgehog commented 8 years ago

Which of logout handlers will be executed? https://github.com/bigcompany/hook.io/blob/master/lib/server/index.js#L144 https://github.com/bigcompany/hook.io/blob/master/lib/server/index.js#L401

Marak commented 8 years ago

Hey, good catch @pyhedgehog!

Definitely a mistake. I believe express will use the last route mapped, so the second handler will apply.

I will fix this for sure. Thank you.