[ckan-changes] [ckan/ckan] 69c631: [#2846] use 403 or 404 when forbidden, not 401

Mon Mar 28 06:22:56 UTC 2016

  Branch: refs/heads/master
  Home:   https://github.com/ckan/ckan
  Commit: 69c6314177c107446550759468ca5ca19c9a88c5
      https://github.com/ckan/ckan/commit/69c6314177c107446550759468ca5ca19c9a88c5
  Author: Ian Ward <ian at excess.org>
  Date:   2016-01-26 (Tue, 26 Jan 2016)

  Changed paths:
    M ckan/config/middleware.py
    M ckan/controllers/admin.py
    M ckan/controllers/group.py
    M ckan/controllers/home.py
    M ckan/controllers/package.py
    M ckan/controllers/revision.py
    M ckan/controllers/tag.py
    M ckan/controllers/user.py
    M ckanext/datapusher/plugin.py

  Log Message:
  -----------
  [#2846] use 403 or 404 when forbidden, not 401


  Commit: a2e96ed0db8357ff0e8965318a5ad95c5d96f695
      https://github.com/ckan/ckan/commit/a2e96ed0db8357ff0e8965318a5ad95c5d96f695
  Author: Ian Ward <ian at excess.org>
  Date:   2016-01-26 (Tue, 26 Jan 2016)

  Changed paths:
    M ckan/tests/controllers/test_admin.py
    M ckan/tests/controllers/test_group.py
    M ckan/tests/controllers/test_organization.py
    M ckan/tests/controllers/test_package.py
    M ckan/tests/controllers/test_user.py
    M ckan/tests/legacy/functional/test_admin.py
    M ckan/tests/legacy/functional/test_package.py
    M ckan/tests/legacy/functional/test_user.py

  Log Message:
  -----------
  [#2846] update tests for 403 responses


  Commit: 068734b526c7edf3995116a5c6b34ffa8a3f690d
      https://github.com/ckan/ckan/commit/068734b526c7edf3995116a5c6b34ffa8a3f690d
  Author: Ian Ward <ian at excess.org>
  Date:   2016-01-26 (Tue, 26 Jan 2016)

  Changed paths:
    M ckan/tests/controllers/test_package.py
    M ckan/tests/controllers/test_user.py
    M ckan/tests/legacy/functional/test_package.py

  Log Message:
  -----------
  [#2846] test fixes (including two that were broken before this change)


  Commit: 71f9892a24df1d2f3797054fa1a40c223d2d84eb
      https://github.com/ckan/ckan/commit/71f9892a24df1d2f3797054fa1a40c223d2d84eb
  Author: Ian Ward <ian at excess.org>
  Date:   2016-01-27 (Wed, 27 Jan 2016)

  Changed paths:
    M ckan/lib/base.py

  Log Message:
  -----------
  [#2846] call IAuthenticator abort() on 403 now


  Commit: c041bbf0978bc430bc9a750290156e03c2bce1a1
      https://github.com/ckan/ckan/commit/c041bbf0978bc430bc9a750290156e03c2bce1a1
  Author: Nigel Babu <nigelbabu at gmail.com>
  Date:   2016-03-28 (Mon, 28 Mar 2016)

  Changed paths:
    M ckan/config/middleware.py
    M ckan/controllers/admin.py
    M ckan/controllers/group.py
    M ckan/controllers/home.py
    M ckan/controllers/package.py
    M ckan/controllers/revision.py
    M ckan/controllers/tag.py
    M ckan/controllers/user.py
    M ckan/lib/base.py
    M ckan/tests/controllers/test_admin.py
    M ckan/tests/controllers/test_group.py
    M ckan/tests/controllers/test_organization.py
    M ckan/tests/controllers/test_package.py
    M ckan/tests/controllers/test_user.py
    M ckan/tests/legacy/functional/test_admin.py
    M ckan/tests/legacy/functional/test_package.py
    M ckan/tests/legacy/functional/test_user.py
    M ckanext/datapusher/plugin.py

  Log Message:
  -----------
  Merge pull request #2846 from ckan/2846-403-forbidden

Use 403 when actions are forbidden, not 401


Compare: https://github.com/ckan/ckan/compare/8dbb6dcacd2b...c041bbf0978b