Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improve catchall routes #24

Open
GrahamGoudeau opened this issue Dec 19, 2016 · 0 comments
Open

Improve catchall routes #24

GrahamGoudeau opened this issue Dec 19, 2016 · 0 comments

Comments

@GrahamGoudeau
Copy link
Owner

For convenience, it would be good to make a new route that matches /api/* route so that, during development, calls to an incorrect api endpoint can return a more sane error response (something like { error: { message: "unknown api endpoint"}}) instead of a blob of HTML.

As another part of this, it would be better to split out the /* and /api/* routes to standalone calls to routeManager.addInsecureRoute() and routeManager.addSecureRoute() respectively (after the calls that take the arrays of routes), so that they don't get accidentally get bundled in with the other routes out of order.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant