Skip to content
This repository was archived by the owner on Oct 8, 2021. It is now read-only.

Untitled#401

Closed
badsyntax wants to merge 2 commits intojquery-archive:masterfrom
badsyntax:master
Closed

Untitled#401
badsyntax wants to merge 2 commits intojquery-archive:masterfrom
badsyntax:master

Conversation

@badsyntax
Copy link

Just some better error handling on the back-end.

@jzaefferer
Copy link
Contributor

Whats the idea of those if-statements? Isn't it to fail silently when those modules are missing? I rather have an error message about some missing module then something not working at all and not telling me whats going on.

@badsyntax
Copy link
Author

Apache gives me an 500 'Internal Server Error', so I had to look in the logs to find the actual configuration error. It was a bit a shlep for me to do this, and I would have preferred the config errors to fail silently. But I do understand others preference on this.

@jzaefferer
Copy link
Contributor

If it fails silently and nothing works, thats worse. Much better would be to output an actual message directly in the browser about the missing module.

@badsyntax
Copy link
Author

Yup makes sense.

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants