Filippo Giunchedi has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/402582 )
Change subject: Upgrade to 1.9 ...................................................................... Upgrade to 1.9 Bug: T169144 Change-Id: I1cb3d133ca3522dd64e52c6b9cb2764863ffc0ad --- M debian/changelog M setup.py M wikimedia_thumbor/handler/images/images.py 3 files changed, 24 insertions(+), 1 deletion(-) Approvals: jenkins-bot: Verified Filippo Giunchedi: Looks good to me, approved diff --git a/debian/changelog b/debian/changelog index 59b5c76..31a1f0f 100644 --- a/debian/changelog +++ b/debian/changelog @@ -1,3 +1,9 @@ +python-thumbor-wikimedia (1.9-1) jessie-wikimedia; urgency=low + + * New upstream release + + -- Gilles Dubuc <gil...@wikimedia.org> Sat, 6 Jan 2018 13:02:00 +0000 + python-thumbor-wikimedia (1.8-1) jessie-wikimedia; urgency=low * New upstream release diff --git a/setup.py b/setup.py index 365555d..5d9cac0 100644 --- a/setup.py +++ b/setup.py @@ -12,7 +12,7 @@ setup( name='wikimedia_thumbor', - version='1.8', + version='1.9', url='https://phabricator.wikimedia.org/diffusion/THMBREXT/', license='MIT', author='Gilles Dubuc, Wikimedia Foundation', diff --git a/wikimedia_thumbor/handler/images/images.py b/wikimedia_thumbor/handler/images/images.py index 3625506..9094809 100644 --- a/wikimedia_thumbor/handler/images/images.py +++ b/wikimedia_thumbor/handler/images/images.py @@ -197,6 +197,11 @@ if hasattr(self.context.config, 'SWIFT_SHARDED_CONTAINERS'): sharded_containers = self.context.config.SWIFT_SHARDED_CONTAINERS + private_containers = [] + + if hasattr(self.context.config, 'SWIFT_PRIVATE_CONTAINERS'): + private_containers = self.context.config.SWIFT_PRIVATE_CONTAINERS + projlang = '-'.join((kw['project'], kw['language'])) original_container = projlang + '-local-public' thumbnail_container = projlang + '-local-thumb' @@ -221,6 +226,8 @@ hashed = md5.new(hashed_name).hexdigest() original_shard1 = hashed[:1] original_shard2 = hashed[:2] + + self.context.private = original_container in private_containers if original_container in sharded_containers: original_container += '.' + original_shard2 @@ -374,6 +381,16 @@ ) return + if self.context.private: + received_secret = self.request.headers.get('X-Swift-Secret', False) + secret = self.context.config.get('SWIFT_PRIVATE_SECRET', False) + if not secret or not received_secret or received_secret != secret: + self._error( + 401, + 'Unauthorized access to private Swift container' + ) + return + xkey = self.set_headers(translated_kw) mc = self.failure_memcache() -- To view, visit https://gerrit.wikimedia.org/r/402582 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I1cb3d133ca3522dd64e52c6b9cb2764863ffc0ad Gerrit-PatchSet: 1 Gerrit-Project: operations/debs/python-thumbor-wikimedia Gerrit-Branch: master Gerrit-Owner: Gilles <gdu...@wikimedia.org> Gerrit-Reviewer: Filippo Giunchedi <fgiunch...@wikimedia.org> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits