jenkins-bot has submitted this change and it was merged.

Change subject: Display better error message for MT processing and other errors.
......................................................................


Display better error message for MT processing and other errors.

* Made same fix for other error logging messages in same file

Change-Id: I63f1237627948ecdb841b3e9a7732ae1853e629e
---
M ContentTranslationService.js
1 file changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Jsahleen: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/ContentTranslationService.js b/ContentTranslationService.js
index e7bf55b..2719eb1 100644
--- a/ContentTranslationService.js
+++ b/ContentTranslationService.js
@@ -72,7 +72,7 @@
                                        error: '' + error
                                } );
                                logger.log( 'error', 'Page %s:%s could not be 
fetched or segmented: (%s)',
-                                       sourceLanguage, title, error );
+                                       sourceLanguage, title, error.toString() 
);
                        }
                        res.send( {
                                sourceLanguage: sourceLanguage,
@@ -136,7 +136,7 @@
                                res.send( 500, {
                                        error: error
                                } );
-                               logger.log( 'error', 'MT processing error: 
(%s)', error );
+                               logger.log( 'error', 'MT processing error: 
(%s)', error.toString() );
                        }
                );
        } );
@@ -170,7 +170,7 @@
                        res.send( 500, {
                                error: error
                        } );
-                       logger.log( 'error', 'Dictionary lookup error: (%s)', 
error );
+                       logger.log( 'error', 'Dictionary lookup error: (%s)', 
error.toString() );
                }
        );
 } );

-- 
To view, visit https://gerrit.wikimedia.org/r/160982
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I63f1237627948ecdb841b3e9a7732ae1853e629e
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/services/cxserver
Gerrit-Branch: master
Gerrit-Owner: KartikMistry <[email protected]>
Gerrit-Reviewer: Jsahleen <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to