[ckan-changes] [ckan/ckan] 2b6c1e: [#2035] Refactor new_authz.check_config_permission...

GitHub noreply at github.com
Wed Nov 19 20:28:15 UTC 2014


  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 2b6c1edb36c54114110c0490d0b1ed5287499565
      https://github.com/ckan/ckan/commit/2b6c1edb36c54114110c0490d0b1ed5287499565
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-07 (Fri, 07 Nov 2014)

  Changed paths:
    M ckan/new_authz.py
    A ckan/new_tests/test_authz.py

  Log Message:
  -----------
  [#2035] Refactor new_authz.check_config_permission to not cache things

Otherwise the values can not be overriden from tests


  Commit: 07f2d0d2bb455eed82d16cb13b189fef659663a9
      https://github.com/ckan/ckan/commit/07f2d0d2bb455eed82d16cb13b189fef659663a9
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-07 (Fri, 07 Nov 2014)

  Changed paths:
    M ckan/new_tests/helpers.py
    M ckan/tests/functional/api/test_user.py

  Log Message:
  -----------
  [#2035] Remove uneeded calls to clear_auth_functions_cache()


  Commit: 3665e3851d125554dd79c57e6f680fdc5dc54d04
      https://github.com/ckan/ckan/commit/3665e3851d125554dd79c57e6f680fdc5dc54d04
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-07 (Fri, 07 Nov 2014)

  Changed paths:
    M ckan/tests/logic/test_auth.py

  Log Message:
  -----------
  [#2035] Fix old auth tests


  Commit: 00b192d489f1f680e8bb8372a23b3e3caf7f689c
      https://github.com/ckan/ckan/commit/00b192d489f1f680e8bb8372a23b3e3caf7f689c
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-10 (Mon, 10 Nov 2014)

  Changed paths:
    M ckan/tests/logic/test_auth.py

  Log Message:
  -----------
  [#2035] Further fixes to auth tests

The old auth tests are terrible in that they depend on each other to
run. Also the auth cache thing masked some bad assumptions.

The whole file should be rewritten as new tests.


  Commit: 7d30f7cc0b8c2879dfa1e1481269f6a55958287b
      https://github.com/ckan/ckan/commit/7d30f7cc0b8c2879dfa1e1481269f6a55958287b
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-10 (Mon, 10 Nov 2014)

  Changed paths:
    M ckanext/resourceproxy/tests/test_proxy.py

  Log Message:
  -----------
  [#2035] Don't use http://foo.bar as a test url

Because `.bar` is being recognized as a gTLD, it points to
`127.0.53.53`, which if you have a web server running will return
localhost (and a 200 status).

More details:

https://stackoverflow.com/questions/25662590/domain-foo-bar-points-127-0-53-53-why


  Commit: 370c811c5ebe2cbca086fa0181bf5787e8c6e254
      https://github.com/ckan/ckan/commit/370c811c5ebe2cbca086fa0181bf5787e8c6e254
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-14 (Fri, 14 Nov 2014)

  Changed paths:
    M ckanext/resourceproxy/tests/test_proxy.py

  Log Message:
  -----------
  [#2035] Improve non-existent URL added on 7d30f7c


  Commit: 99e5ee0d6c8499a262038aa7a724590298b5d81f
      https://github.com/ckan/ckan/commit/99e5ee0d6c8499a262038aa7a724590298b5d81f
  Author: amercader <amercadero at gmail.com>
  Date:   2014-11-14 (Fri, 14 Nov 2014)

  Changed paths:
    M ckan/new_authz.py

  Log Message:
  -----------
  [#2035] Split all whitespace when parsing roles_that_cascade_to_sub_groups

Use split() rather than split(' ')


  Commit: 109fdea682f175e15b8e2a78a09c95fcebd6a35b
      https://github.com/ckan/ckan/commit/109fdea682f175e15b8e2a78a09c95fcebd6a35b
  Author: Ian Ward <ian at excess.org>
  Date:   2014-11-19 (Wed, 19 Nov 2014)

  Changed paths:
    M ckan/new_authz.py
    M ckan/new_tests/helpers.py
    A ckan/new_tests/test_authz.py
    M ckan/tests/functional/api/test_user.py
    M ckan/tests/logic/test_auth.py
    M ckanext/resourceproxy/tests/test_proxy.py

  Log Message:
  -----------
  Merge branch '2035-remove-auth-permissions-cache'


Compare: https://github.com/ckan/ckan/compare/0ddd6fda6103...109fdea682f1


More information about the ckan-changes mailing list