bananaml / potassium

An HTTP serving framework by Banana
Apache License 2.0
97 stars 9 forks source link

raise Exception on endpoint collision + handle empty init #27

Closed erik-dunteman closed 1 year ago

erik-dunteman commented 1 year ago

What is this?

We raise an Exception when routes are repeated

plus a sidequest

Why?

Both endpoint collision and misuse of init were leading to user confusion

How did you test it works without regressions?

Local tests with multiple endpoints

If this is a new feature what may a critical error look like?

not new feature, but technically a breaking change since we raise exceptions in these cases rather than silently fail.