audit-logs: implement enum sources that should be re-used.

This commit is contained in:
Marcin Kuzminski 2017-06-14 11:51:46 +02:00
parent c36f3f7abd
commit dbcc488137
3 changed files with 9 additions and 3 deletions

View file

@ -1181,7 +1181,8 @@ def delete_repo(request, apiuser, repoid, forks=Optional('')):
audit_logger.store(
action='repo.delete',
action_data={'repo_data': repo_data, 'source': 'api_call'},
action_data={'repo_data': repo_data,
'source': audit_logger.SOURCE_API},
user=apiuser, repo=repo, commit=False)
ScmModel().mark_for_invalidation(repo_name, delete=True)

View file

@ -101,7 +101,8 @@ class RepoSettingsView(RepoAppView):
repo_name=self.db_repo.repo_name)
audit_logger.store(
action='repo.delete',
action_data={'repo_data': repo_data, 'source': 'web_action'},
action_data={'repo_data': repo_data,
'source': audit_logger.SOURCE_WEB},
user=self._rhodecode_user, repo=repo, commit=False)
ScmModel().mark_for_invalidation(self.db_repo_name, delete=True)

View file

@ -44,6 +44,9 @@ ACTIONS = {
'repo.archive.download': {},
}
SOURCE_WEB = 'source_web'
SOURCE_API = 'source_api'
class UserWrap(object):
"""
@ -114,7 +117,8 @@ def store(
# repo action, when we know and have the repository object already
audit_logger.store(
action='repo.delete',
user=audit_logger.UserWrap(username='itried-login', ip_addr='8.8.8.8'),
action_data={'source': audit_logger.SOURCE_WEB, },
user=self._rhodecode_user,
repo=repo_object)
# without an user ?