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

Change subject: Add length check to YandexWebService
......................................................................


Add length check to YandexWebService

Change-Id: I229d30aa20c027e9642be6557d28122ba7c6cdde
---
M webservices/YandexWebService.php
1 file changed, 5 insertions(+), 0 deletions(-)

Approvals:
  KartikMistry: Looks good to me, approved
  Nemo bis: Looks good to me, but someone else must approve
  jenkins-bot: Verified



diff --git a/webservices/YandexWebService.php b/webservices/YandexWebService.php
index 3f9a313..9a4a87d 100644
--- a/webservices/YandexWebService.php
+++ b/webservices/YandexWebService.php
@@ -55,6 +55,11 @@
                        throw new TranslationWebServiceException( 'API key is 
not set' );
                }
 
+               # http://api.yandex.com/translate/doc/dg/reference/translate.xml
+               if ( strlen( $text ) > 10000 ) {
+                       throw new TranslationWebServiceException( 'Source text 
too long' );
+               }
+
                $service = $this->service;
 
                $text = trim( $text );

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I229d30aa20c027e9642be6557d28122ba7c6cdde
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Translate
Gerrit-Branch: master
Gerrit-Owner: Nikerabbit <[email protected]>
Gerrit-Reviewer: KartikMistry <[email protected]>
Gerrit-Reviewer: Nemo bis <[email protected]>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to