Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

check_access logging untidy #2165

Closed
davidread opened this issue Dec 29, 2014 · 0 comments
Closed

check_access logging untidy #2165

davidread opened this issue Dec 29, 2014 · 0 comments
Assignees
@davidread

Comments

@davidread
Copy link
Contributor

When we call check_access it is a bit annoying that it takes two logs to see what happened:

2014-12-29 10:41:28,743 DEBUG [ckan.logic] check access - user None, action get_site_user
2014-12-29 10:41:28,743 DEBUG [ckan.logic] Access OK.

It would be better if it was just one.
davidread pushed a commit that referenced this issue Dec 29, 2014
[#2165] Only one log message now on check_access.
b80d87c
@davidread davidread mentioned this issue Dec 29, 2014
Merged
davidread pushed a commit that referenced this issue Dec 29, 2014
[#2165] Cope with unicode in the NotAuthorized message. Document exis…
49aae60 
…ting unicode behaviour with tests.
davidread pushed a commit that referenced this issue Dec 29, 2014
[#2165] PEP8.
99f9c03
rossjones added a commit that referenced this issue Dec 31, 2014
@rossjones
Merge pull request #2166 from ckan/2165-check-access-log
0e72f17 
[#2165] Only one log message now on check_access.
@davidread davidread self-assigned this Jan 6, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@davidread davidread

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@davidread