Gilles has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/386348 )

Change subject: Upgrade to 1.7
......................................................................

Upgrade to 1.7

Bug: T178974
Bug: T178072
Change-Id: I51f46f0c46b7a5b3a884888ce0a0130a4331be58
---
M debian/changelog
M setup.py
M tests/integration/test_images_handler.py
M wikimedia_thumbor/engine/imagemagick/imagemagick.py
M wikimedia_thumbor/handler/images/images.py
5 files changed, 25 insertions(+), 3 deletions(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/operations/debs/python-thumbor-wikimedia 
refs/changes/48/386348/1

diff --git a/debian/changelog b/debian/changelog
index 73eae37..0a00260 100644
--- a/debian/changelog
+++ b/debian/changelog
@@ -1,3 +1,9 @@
+python-thumbor-wikimedia (1.7-1) jessie-wikimedia; urgency=low
+
+  * New upstream release
+
+ -- Gilles Dubuc <gil...@wikimedia.org>  Wed, 25 Oct 2017 10:30:00 +0000
+
 python-thumbor-wikimedia (1.6-1) jessie-wikimedia; urgency=low
 
   * New upstream release
diff --git a/setup.py b/setup.py
index 188d79f..57716a5 100644
--- a/setup.py
+++ b/setup.py
@@ -12,7 +12,7 @@
 
 setup(
     name='wikimedia_thumbor',
-    version='1.6',
+    version='1.7',
     url='https://phabricator.wikimedia.org/diffusion/THMBREXT/',
     license='MIT',
     author='Gilles Dubuc, Wikimedia Foundation',
diff --git a/tests/integration/test_images_handler.py 
b/tests/integration/test_images_handler.py
index 025b0ff..135ffed 100644
--- a/tests/integration/test_images_handler.py
+++ b/tests/integration/test_images_handler.py
@@ -209,6 +209,18 @@
             'format(png):lang(fr)',
             'inline;filename*=UTF-8\'\'1Mcolors.svg.png'
         )
+        self.run_and_check_headers(
+            
'/wikipedia/en/thumb/d/d3/1Mcolors.svg/langzh-hans-400px-1Mcolors.svg.png',
+            'File:1Mcolors.svg',
+            'wikipedia-en-local-public.d3',
+            'd/d3/1Mcolors.svg',
+            'wikipedia-en-local-thumb.d3',
+            'thumbor/d/d3/1Mcolors.svg/langzh-hans-400px-1Mcolors.svg.png',
+            '400',
+            
'http://swifthost/swift/v1/api/path/wikipedia-en-local-public.d3/d/d3/1Mcolors.svg',
+            'format(png):lang(zh-hans)',
+            'inline;filename*=UTF-8\'\'1Mcolors.svg.png'
+        )
 
     def test_seek(self):
         self.run_and_check_headers(
diff --git a/wikimedia_thumbor/engine/imagemagick/imagemagick.py 
b/wikimedia_thumbor/engine/imagemagick/imagemagick.py
index 4b041cd..ee1bdaa 100644
--- a/wikimedia_thumbor/engine/imagemagick/imagemagick.py
+++ b/wikimedia_thumbor/engine/imagemagick/imagemagick.py
@@ -11,6 +11,7 @@
 
 # ImageMagick engine
 
+import logging
 from tempfile import NamedTemporaryFile
 from pyexiv2 import ImageMetadata
 
@@ -123,7 +124,10 @@
 
         metadata = ImageMetadata(input_temp_file.name)
         try:
+            # T178072 pyexviv2 writes to stderr even if the exception is caught
+            logging.disable(logging.ERROR)
             metadata.read()
+            logging.disable(logging.NOTSET)
             if 'Exif.Image.Orientation' in metadata.exif_keys:
                 # Distinctive key name to avoid colliding with 
EXIF_FIELDS_TO_KEEP
                 self.exif['Pyexiv2Orientation'] = 
metadata.get('Exif.Image.Orientation').value
diff --git a/wikimedia_thumbor/handler/images/images.py 
b/wikimedia_thumbor/handler/images/images.py
index 9940305..3625506 100644
--- a/wikimedia_thumbor/handler/images/images.py
+++ b/wikimedia_thumbor/handler/images/images.py
@@ -132,11 +132,11 @@
             r'(?:(?P<lossy>lossy-))?'
             r'(?:(?P<lossless>lossless-))?'
             r'(?:page(?P<page>\d+)-)?'
-            r'(?:lang(?P<lang>[a-zA-Z]+)-)?'
+            r'(?:lang(?P<lang>[0-9a-zA-Z-]+)-)?'
             r'(?P<width>\d+)px-'
             r'(?:(?:seek=|seek%3D)(?P<seek>\d+)-)?'
             r'(?P<end>[^/]+)'
-            r'\.(?P<format>[a-zA-Z]+)'
+            r'\.(?P<format>[0-9a-zA-Z]+)'
         )
 
     def generate_save_swift_path(self, kw):

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I51f46f0c46b7a5b3a884888ce0a0130a4331be58
Gerrit-PatchSet: 1
Gerrit-Project: operations/debs/python-thumbor-wikimedia
Gerrit-Branch: master
Gerrit-Owner: Gilles <gdu...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to