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.

Sign up for GitHub

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

Jump to bottom

Refactor error handling #3

Open
nearwood opened this issue Mar 27, 2016 · 1 comment
Open

Refactor error handling #3

nearwood opened this issue Mar 27, 2016 · 1 comment
Labels
help wanted

Comments

@nearwood
Copy link
Owner

Skip the try catch (unless we expect errors and can continue), and use the error_handler callback for major problems (and send a 500 response, for example).
@nearwood
Copy link
Owner Author

nearwood commented Oct 31, 2019 •
edited
Loading

Also, remove all the html from the log file output.

nearwood added a commit that referenced this issue Nov 5, 2019
@nearwood
Improve parse errors #3, and avoid cache if it fails #31
e632ddf
nearwood added a commit that referenced this issue Nov 5, 2019
@nearwood
Merge pull request #32 from nearwood/master
931aedf 
Improve parse errors #3, and avoid cache if it fails #31
nearwood added a commit that referenced this issue Nov 5, 2019
@nearwood
Improve parse errors #3, and avoid cache if it fails #31
306b671
@nearwood nearwood mentioned this issue Nov 5, 2019
Merged
nearwood added a commit that referenced this issue Nov 5, 2019
@nearwood
Error handling improvements (#33)
b6bfa88 
* Improve parse errors #3, and avoid cache if it fails #31

* add version string to deploy script
nearwood added a commit that referenced this issue Nov 7, 2019
@nearwood
HTML Semantics, SEO, fix caching of failed parses, text content update (
3ea73f9 
#40)

* Improve parse errors #3, and avoid cache if it fails #31

* add version string to deploy script

* Fix confusing getMSQ function name

* Update splash

* Fix invalid html #37

* Fix linter warnings

* Fix invalid HTML

* Fix html validation select element

* Add img alt text

* Add favicon #38

* Add SRI #39

* Add button image, tweak CSS

* Fix download header and add download hint #30

* Cleanup header

* add async to scripts

* Add open graph stuff #35 #36

* Fix async scripts

* Header color

* Fix travis deploying PRs
nearwood added a commit that referenced this issue Nov 7, 2019
@nearwood
Fix CD (#41)
a2c852b 
* Improve parse errors #3, and avoid cache if it fails #31

* add version string to deploy script

* Fix confusing getMSQ function name

* Update splash

* Fix invalid html #37

* Fix linter warnings

* Fix invalid HTML

* Fix html validation select element

* Add img alt text

* Add favicon #38

* Add SRI #39

* Add button image, tweak CSS

* Fix download header and add download hint #30

* Cleanup header

* add async to scripts

* Add open graph stuff #35 #36

* Fix async scripts

* Header color

* Fix travis deploying PRs

* Fix deploy hopefully

* Fix deploy hopefully

* travis ci docs aren't great
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
help wanted
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@nearwood