[ckan-changes] [ckan/ckan] ea4ceb: Upgrade werkzeug

Fri Sep 6 13:31:39 UTC 2019

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: ea4ceb404d826a48b8e66872640e6ffffe1ee0f7
      https://github.com/ckan/ckan/commit/ea4ceb404d826a48b8e66872640e6ffffe1ee0f7
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-08-21 (Wed, 21 Aug 2019)

  Changed paths:
    M requirements.in
    M requirements.txt

  Log Message:
  -----------
  Upgrade werkzeug

  Commit: b2f25f966660eed95d2e200379c1cadb1719423b
      https://github.com/ckan/ckan/commit/b2f25f966660eed95d2e200379c1cadb1719423b
  Author: David Read <david.read at hackneyworkshop.com>
  Date:   2019-08-22 (Thu, 22 Aug 2019)

  Changed paths:
    M ckan/tests/controllers/test_home.py
    M ckan/tests/helpers.py
    M ckan/tests/legacy/functional/test_preview_interface.py
    M ckanext/datastore/tests/test_info.py
    M ckanext/multilingual/tests/test_multilingual_plugin.py

  Log Message:
  -----------
  Test app urls need to be str type

With this upgrade of werkzeug, url_for now returns unicode, rather than a str. I'm not clear where that happens, but flask uses lots of bits of werkzeug underneath.

The PATH_INFO exception occurs because WSGI requires environ.PATH_INFO to be a string (bytes, not unicode). This is being checked by webtest.lint (the newest version does the same check). PATH_INFO is the part of the WSGI request storing the URL's path. WebOb needs to save it as str, but our old version, webob.request stores it in whatever encoding we give it, so we need to specify the URL already encoded.

I'm not 100% sure that UTF8 is the correct encoding - it might need url (percent) encoding, but for these tests it doesn't make any difference.

  Commit: 06376d903e0fc04d8a14b52a3e4816e5b431324e
      https://github.com/ckan/ckan/commit/06376d903e0fc04d8a14b52a3e4816e5b431324e
  Author: Adrià Mercader <amercadero at gmail.com>
  Date:   2019-09-06 (Fri, 06 Sep 2019)

  Changed paths:
    M ckan/tests/controllers/test_home.py
    M ckan/tests/helpers.py
    M ckan/tests/legacy/functional/test_preview_interface.py
    M ckanext/datastore/tests/test_info.py
    M ckanext/multilingual/tests/test_multilingual_plugin.py
    M requirements.in
    M requirements.txt

  Log Message:
  -----------
  Merge pull request #4945 from ckan/werkzeug-0.15

Upgrade werkzeug

Compare: https://github.com/ckan/ckan/compare/e63a4efd0d44...06376d903e0f