[ckan-changes] [ckan/ckan] 00b192: [#2035] Further fixes to auth tests
GitHub
noreply at github.com
Mon Nov 10 17:15:02 UTC 2014
Branch: refs/heads/2035-remove-auth-permissions-cache
Home: https://github.com/ckan/ckan
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
Compare: https://github.com/ckan/ckan/compare/3665e3851d12...7d30f7cc0b8c
More information about the ckan-changes
mailing list