KartikMistry has uploaded a new change for review.
https://gerrit.wikimedia.org/r/316610
Change subject: Apertium: Reject translation of > 10000 chars and log it
......................................................................
Apertium: Reject translation of > 10000 chars and log it
Change-Id: If98d770335528dda338d40d36a533061cb90e5ed
---
M mt/Apertium.js
1 file changed, 6 insertions(+), 0 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/services/cxserver
refs/changes/10/316610/1
diff --git a/mt/Apertium.js b/mt/Apertium.js
index 8651d0f..25893be 100644
--- a/mt/Apertium.js
+++ b/mt/Apertium.js
@@ -3,6 +3,7 @@
var preq = require( 'preq' ),
util = require( 'util' ),
MTClient = require( './MTClient.js' ),
+ BBPromise = require( 'bluebird' ),
apertiumLangMapping = require( './Apertium.languagenames.json' );
function Apertium( options ) {
@@ -25,6 +26,11 @@
* @return {BBPromise} promise: Target language text
*/
Apertium.prototype.translateText = function ( sourceLang, targetLang,
sourceText ) {
+
+ if ( sourceText.length > 10000 ) {
+ return BBPromise.reject( new Error( 'Source text too long' ) );
+ }
+
return preq.post( {
uri: this.conf.mt.apertium.api + '/translate',
body: {
--
To view, visit https://gerrit.wikimedia.org/r/316610
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: If98d770335528dda338d40d36a533061cb90e5ed
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/services/cxserver
Gerrit-Branch: master
Gerrit-Owner: KartikMistry <kartik.mis...@gmail.com>
_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
