[Zope-Checkins] SVN: Zope/branches/2.13/ Revert r127986 - tempstorage is an actual dependency of Products.TemporaryStorage
Log message for revision 127987: Revert r127986 - tempstorage is an actual dependency of Products.TemporaryStorage Changed: U Zope/branches/2.13/buildout.cfg U Zope/branches/2.13/setup.py -=- Modified: Zope/branches/2.13/buildout.cfg === --- Zope/branches/2.13/buildout.cfg 2012-10-13 19:45:24 UTC (rev 127986) +++ Zope/branches/2.13/buildout.cfg 2012-10-14 09:19:32 UTC (rev 127987) @@ -30,7 +30,7 @@ import sys import warnings if sys.version_info = (2, 7): warnings.simplefilter('default') -eggs = Zope2 [test] +eggs = Zope2 [scripts] Modified: Zope/branches/2.13/setup.py === --- Zope/branches/2.13/setup.py 2012-10-13 19:45:24 UTC (rev 127986) +++ Zope/branches/2.13/setup.py 2012-10-14 09:19:32 UTC (rev 127987) @@ -65,6 +65,7 @@ 'initgroups', 'pytz', 'setuptools', + 'tempstorage', 'transaction', 'zdaemon', 'zExceptions', @@ -112,9 +113,7 @@ 'Products.PythonScripts', 'Products.StandardCacheManagers', ] + additional_install_requires, -extras_require={ - 'test': ['tempstorage'], -}, + include_package_data=True, zip_safe=False, entry_points={ ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins
[Zope-Checkins] SVN: Zope/trunk/ Revert r127985 - tempstorage is an actual dependency of Products.TemporaryFolder
Log message for revision 127988: Revert r127985 - tempstorage is an actual dependency of Products.TemporaryFolder Changed: U Zope/trunk/buildout.cfg U Zope/trunk/setup.py -=- Modified: Zope/trunk/buildout.cfg === --- Zope/trunk/buildout.cfg 2012-10-14 09:19:32 UTC (rev 127987) +++ Zope/trunk/buildout.cfg 2012-10-14 09:21:40 UTC (rev 127988) @@ -31,7 +31,7 @@ import sys import warnings if sys.version_info = (2, 7): warnings.simplefilter('default') -eggs = Zope2 [test] +eggs = Zope2 [scripts] Modified: Zope/trunk/setup.py === --- Zope/trunk/setup.py 2012-10-14 09:19:32 UTC (rev 127987) +++ Zope/trunk/setup.py 2012-10-14 09:21:40 UTC (rev 127988) @@ -65,6 +65,7 @@ 'initgroups', 'pytz', 'setuptools', + 'tempstorage', 'transaction', 'zdaemon', 'zExceptions', @@ -105,9 +106,7 @@ 'zope.traversing', 'zope.viewlet', ] + additional_install_requires, -extras_require={ - 'test': ['tempstorage'], -}, + include_package_data=True, zip_safe=False, entry_points={ ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins
[Zope-Checkins] SVN: Zope/trunk/versions.cfg tempstorage = 2.12.2
Log message for revision 127995: tempstorage = 2.12.2 Changed: U Zope/trunk/versions.cfg -=- Modified: Zope/trunk/versions.cfg === --- Zope/trunk/versions.cfg 2012-10-14 12:01:46 UTC (rev 127994) +++ Zope/trunk/versions.cfg 2012-10-14 12:02:49 UTC (rev 127995) @@ -19,7 +19,7 @@ Products.ZCatalog = 3.0a2 Products.ZCTextIndex = 2.13.3 Record = 2.13.0 -tempstorage = 2.12.1 +tempstorage = 2.12.2 zExceptions = 2.13.0 zLOG = 2.12.0 ZopeUndo = 2.12.0 ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins
[Zope-Checkins] SVN: Zope/branches/2.12/ LP #930812: Scrub headers a bit more.
Log message for revision 127996: LP #930812: Scrub headers a bit more. Changed: U Zope/branches/2.12/doc/CHANGES.rst U Zope/branches/2.12/src/ZPublisher/HTTPResponse.py U Zope/branches/2.12/src/ZPublisher/tests/testHTTPResponse.py -=- Modified: Zope/branches/2.12/doc/CHANGES.rst === --- Zope/branches/2.12/doc/CHANGES.rst 2012-10-14 12:02:49 UTC (rev 127995) +++ Zope/branches/2.12/doc/CHANGES.rst 2012-10-14 12:38:50 UTC (rev 127996) @@ -8,6 +8,7 @@ 2.12.26 (unreleased) +- LP #930812: Scrub headers a bit more. 2.12.25 (2012-09-18) @@ -15,7 +16,6 @@ - Explicitly declared ZTUtils APIs as public (repairs breakages in apps following fix for LP #1047318). - 2.12.24 (2012-09-09) Modified: Zope/branches/2.12/src/ZPublisher/HTTPResponse.py === --- Zope/branches/2.12/src/ZPublisher/HTTPResponse.py 2012-10-14 12:02:49 UTC (rev 127995) +++ Zope/branches/2.12/src/ZPublisher/HTTPResponse.py 2012-10-14 12:38:50 UTC (rev 127996) @@ -125,7 +125,7 @@ if otherTypes: uncompressableMimeMajorTypes += tuple(otherTypes.split(',')) -_CRLF = re.compile(r'\r[\n]?') +_CRLF = re.compile(r'[\r\n]') def _scrubHeader(name, value): return ''.join(_CRLF.split(str(name))), ''.join(_CRLF.split(str(value))) Modified: Zope/branches/2.12/src/ZPublisher/tests/testHTTPResponse.py === --- Zope/branches/2.12/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:02:49 UTC (rev 127995) +++ Zope/branches/2.12/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:38:50 UTC (rev 127996) @@ -303,6 +303,15 @@ 'Set-Cookie: ' 'violation=http://www.ietf.org/rfc/rfc2616.txt\r\n') +def test_setHeader_drops_LF(self): +# Some browsers accept \n in place of \n\r to separate headers, +# so we scrub it too. +response = self._makeOne() +response.setHeader('Location', + 'http://www.ietf.org/rfc/\nrfc2616.txt') +self.assertEqual(response.headers['location'], + 'http://www.ietf.org/rfc/rfc2616.txt') + def test_setBody_compression_vary(self): # Vary header should be added here response = self._makeOne() ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins
[Zope-Checkins] SVN: Zope/branches/2.13/ LP #930812: Scrub headers a bit more.
Log message for revision 127997: LP #930812: Scrub headers a bit more. Changed: U Zope/branches/2.13/doc/CHANGES.rst U Zope/branches/2.13/src/ZPublisher/HTTPResponse.py U Zope/branches/2.13/src/ZPublisher/tests/testHTTPResponse.py -=- Modified: Zope/branches/2.13/doc/CHANGES.rst === --- Zope/branches/2.13/doc/CHANGES.rst 2012-10-14 12:38:50 UTC (rev 127996) +++ Zope/branches/2.13/doc/CHANGES.rst 2012-10-14 12:43:43 UTC (rev 127997) @@ -8,6 +8,8 @@ 2.13.19 (unreleased) +- LP #930812: Scrub headers a bit more. + - Updated distributions: - tempstorage = 2.12.2 Modified: Zope/branches/2.13/src/ZPublisher/HTTPResponse.py === --- Zope/branches/2.13/src/ZPublisher/HTTPResponse.py 2012-10-14 12:38:50 UTC (rev 127996) +++ Zope/branches/2.13/src/ZPublisher/HTTPResponse.py 2012-10-14 12:43:43 UTC (rev 127997) @@ -128,7 +128,7 @@ if otherTypes: uncompressableMimeMajorTypes += tuple(otherTypes.split(',')) -_CRLF = re.compile(r'\r[\n]?') +_CRLF = re.compile(r'[\r\n]') def _scrubHeader(name, value): return ''.join(_CRLF.split(str(name))), ''.join(_CRLF.split(str(value))) Modified: Zope/branches/2.13/src/ZPublisher/tests/testHTTPResponse.py === --- Zope/branches/2.13/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:38:50 UTC (rev 127996) +++ Zope/branches/2.13/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:43:43 UTC (rev 127997) @@ -431,6 +431,15 @@ ('Set-Cookie', 'violation=http://www.ietf.org/rfc/rfc2616.txt;')]) +def test_setHeader_drops_LF(self): +# Some browsers accept \n in place of \n\r to separate headers, +# so we scrub it too. +response = self._makeOne() +response.setHeader('Location', + 'http://www.ietf.org/rfc/\nrfc2616.txt') +self.assertEqual(response.headers['location'], + 'http://www.ietf.org/rfc/rfc2616.txt') + def test_appendHeader_no_existing(self): response = self._makeOne() response.appendHeader('foo', 'foo') ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins
[Zope-Checkins] SVN: Zope/trunk/src/ZPublisher/ - LP #930812: Scrub headers a bit more.
Log message for revision 127998: - LP #930812: Scrub headers a bit more. Changed: U Zope/trunk/src/ZPublisher/HTTPResponse.py U Zope/trunk/src/ZPublisher/tests/testHTTPResponse.py -=- Modified: Zope/trunk/src/ZPublisher/HTTPResponse.py === --- Zope/trunk/src/ZPublisher/HTTPResponse.py 2012-10-14 12:43:43 UTC (rev 127997) +++ Zope/trunk/src/ZPublisher/HTTPResponse.py 2012-10-14 12:46:17 UTC (rev 127998) @@ -128,7 +128,7 @@ if otherTypes: uncompressableMimeMajorTypes += tuple(otherTypes.split(',')) -_CRLF = re.compile(r'\r[\n]?') +_CRLF = re.compile(r'[\r\n]') def _scrubHeader(name, value): return ''.join(_CRLF.split(str(name))), ''.join(_CRLF.split(str(value))) Modified: Zope/trunk/src/ZPublisher/tests/testHTTPResponse.py === --- Zope/trunk/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:43:43 UTC (rev 127997) +++ Zope/trunk/src/ZPublisher/tests/testHTTPResponse.py 2012-10-14 12:46:17 UTC (rev 127998) @@ -409,6 +409,15 @@ self.assertEqual(response.headers['location'], 'http://www.ietf.org/rfc/rfc2616.txt') +def test_setHeader_drops_LF(self): +# Some browsers accept \n in place of \n\r to separate headers, +# so we scrub it too. +response = self._makeOne() +response.setHeader('Location', + 'http://www.ietf.org/rfc/\nrfc2616.txt') +self.assertEqual(response.headers['location'], + 'http://www.ietf.org/rfc/rfc2616.txt') + def test_appendHeader_no_existing(self): response = self._makeOne() response.appendHeader('foo', 'foo') ___ Zope-Checkins maillist - Zope-Checkins@zope.org https://mail.zope.org/mailman/listinfo/zope-checkins