[ckan-changes] [ckan/ckan] f353e2: [#2037] Make resource_create auth work against pac...

GitHub noreply at github.com
Tue Feb 10 14:12:24 UTC 2015


  Branch: refs/heads/release-v2.0.6
  Home:   https://github.com/ckan/ckan
  Commit: f353e26ac5d74bc8c088fbffcaa50352f86c6497
      https://github.com/ckan/ckan/commit/f353e26ac5d74bc8c088fbffcaa50352f86c6497
  Author: amercader <amercadero at gmail.com>
  Date:   2015-02-10 (Tue, 10 Feb 2015)

  Changed paths:
    M ckan/logic/auth/create.py

  Log Message:
  -----------
  [#2037] Make resource_create auth work against package_update

Right now it was deferring to package_create, which doesn't make much
sense. Basically if you can update this particular dataset, you should
be able to add a resource to it. Added auth tests.

Also fixed the authorization for resource views creation. This is being
fully tested on #1852

Conflicts:
	ckan/logic/auth/create.py
	ckan/new_tests/logic/auth/test_create.py


  Commit: 2389401b6e71224700be6a0aeb4eff471821ac13
      https://github.com/ckan/ckan/commit/2389401b6e71224700be6a0aeb4eff471821ac13
  Author: amercader <amercadero at gmail.com>
  Date:   2015-02-10 (Tue, 10 Feb 2015)

  Changed paths:
    M ckan/logic/action/create.py

  Log Message:
  -----------
  [#2037] Don't remove package_id before resource_create auth check

I didn't find any good reason as to why it was being done, and it
prevents `resource_create` auth to check authorization against the
dataset.

Added action tests

Conflicts:
	ckan/logic/action/create.py
	ckan/new_tests/logic/action/test_create.py


Compare: https://github.com/ckan/ckan/compare/3c45edc0f1e0...2389401b6e71


More information about the ckan-changes mailing list