Skip to content
This repository has been archived by the owner on Mar 22, 2018. It is now read-only.

Commit

Permalink
Add more error handling.
Browse files Browse the repository at this point in the history
Add error handling for connection errors like "SSL Handshake failed".
  • Loading branch information
smeinecke committed Jan 9, 2015
1 parent 1658097 commit 402d688
Showing 1 changed file with 6 additions and 3 deletions.
9 changes: 6 additions & 3 deletions scripts/rasterizer.js
Expand Up @@ -6,7 +6,7 @@
*
* This starts an HTTP server waiting for screenshot requests
*/
var basePath = phantom.args[0] || '/tmp/';
var basePath = phantom.args[0] || '/tmp/';

var port = phantom.args[1] || 3001;

Expand Down Expand Up @@ -49,7 +49,7 @@ server = require('webserver').create();
* All settings of the WebPage object can also be set using headers, e.g.:
* javascriptEnabled: false
* userAgent: Mozilla/5.0 (iPhone; U; CPU like Mac OS X; en) AppleWebKit/420+
*/
*/
service = server.listen(port, function(request, response) {
if (request.url == '/healthCheck') {
response.statusCode = 200;
Expand All @@ -66,6 +66,9 @@ service = server.listen(port, function(request, response) {
var url = request.headers.url;
var path = basePath + (request.headers.filename || (url.replace(new RegExp('https?://'), '').replace(/\//g, '.') + '.png'));
var page = new WebPage();
page.onResourceError = function(resourceError) {
page.error_reason = resourceError.errorString;
};
var delay = request.headers.delay || 0;
try {
page.viewportSize = {
Expand Down Expand Up @@ -95,7 +98,7 @@ service = server.listen(port, function(request, response) {
response.close();
}, delay);
} else {
response.write('Error: Url returned status ' + status + "\n");
response.write('Error: Url returned status ' + status + ' - ' + page.error_reason + "\n");
page.release();
response.close();
}
Expand Down

0 comments on commit 402d688

Please sign in to comment.