mirror of
https://github.com/haiwen/seahub.git
synced 2025-09-24 04:48:03 +00:00
[folder-perm] log error
This commit is contained in:
@@ -2289,11 +2289,12 @@ def set_user_folder_perm(request, repo_id):
|
|||||||
if not user_folder_perm:
|
if not user_folder_perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.add_folder_user_perm(repo_id, path, perm, user)
|
seafile_api.add_folder_user_perm(repo_id, path, perm, user)
|
||||||
send_perm_audit_msg('add-repo-perm', request.user.username, user, \
|
send_perm_audit_msg('add-repo-perm', request.user.username,
|
||||||
repo_id, path, perm)
|
user, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Already add this folder permission')}),
|
return HttpResponse(json.dumps({"error": _('Already add this folder permission')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
@@ -2302,11 +2303,12 @@ def set_user_folder_perm(request, repo_id):
|
|||||||
if user_folder_perm and user_folder_perm != perm:
|
if user_folder_perm and user_folder_perm != perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.set_folder_user_perm(repo_id, path, perm, user)
|
seafile_api.set_folder_user_perm(repo_id, path, perm, user)
|
||||||
send_perm_audit_msg('modify-repo-perm', request.user.username, user, \
|
send_perm_audit_msg('modify-repo-perm', request.user.username,
|
||||||
repo_id, path, perm)
|
user, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Wrong folder permission')}),
|
return HttpResponse(json.dumps({"error": _('Wrong folder permission')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
@@ -2315,14 +2317,16 @@ def set_user_folder_perm(request, repo_id):
|
|||||||
if user_folder_perm:
|
if user_folder_perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.rm_folder_user_perm(repo_id, path, user)
|
seafile_api.rm_folder_user_perm(repo_id, path, user)
|
||||||
send_perm_audit_msg('delete-repo-perm', request.user.username, user, \
|
send_perm_audit_msg('delete-repo-perm', request.user.username,
|
||||||
repo_id, path, perm)
|
user, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Please add folder permission first')}),
|
return HttpResponse(json.dumps({"error": _('Please add folder permission first')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
|
|
||||||
return HttpResponse(json.dumps({'success': True}),
|
return HttpResponse(json.dumps({'success': True}),
|
||||||
content_type=content_type)
|
content_type=content_type)
|
||||||
|
|
||||||
@@ -2358,11 +2362,12 @@ def set_group_folder_perm(request, repo_id):
|
|||||||
if not group_folder_perm:
|
if not group_folder_perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.add_folder_group_perm(repo_id, path, perm, group_id)
|
seafile_api.add_folder_group_perm(repo_id, path, perm, group_id)
|
||||||
send_perm_audit_msg('add-repo-perm', request.user.username, \
|
send_perm_audit_msg('add-repo-perm', request.user.username,
|
||||||
group_id, repo_id, path, perm)
|
group_id, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Already add this folder permission')}),
|
return HttpResponse(json.dumps({"error": _('Already add this folder permission')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
@@ -2371,11 +2376,12 @@ def set_group_folder_perm(request, repo_id):
|
|||||||
if group_folder_perm and group_folder_perm != perm:
|
if group_folder_perm and group_folder_perm != perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.set_folder_group_perm(repo_id, path, perm, group_id)
|
seafile_api.set_folder_group_perm(repo_id, path, perm, group_id)
|
||||||
send_perm_audit_msg('modify-repo-perm', request.user.username, \
|
send_perm_audit_msg('modify-repo-perm', request.user.username,
|
||||||
group_id, repo_id, path, perm)
|
group_id, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Wrong folder permission')}),
|
return HttpResponse(json.dumps({"error": _('Wrong folder permission')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
@@ -2384,11 +2390,12 @@ def set_group_folder_perm(request, repo_id):
|
|||||||
if group_folder_perm:
|
if group_folder_perm:
|
||||||
try:
|
try:
|
||||||
seafile_api.rm_folder_group_perm(repo_id, path, group_id)
|
seafile_api.rm_folder_group_perm(repo_id, path, group_id)
|
||||||
send_perm_audit_msg('delete-repo-perm', request.user.username, \
|
send_perm_audit_msg('delete-repo-perm', request.user.username,
|
||||||
group_id, repo_id, path, perm)
|
group_id, repo_id, path, perm)
|
||||||
except SearpcError, e:
|
except SearpcError as e:
|
||||||
return HttpResponse(json.dumps({"error": e.msg}), status=500,
|
logger.error(e)
|
||||||
content_type=content_type)
|
return HttpResponse(json.dumps({"error": _('Operation failed')}),
|
||||||
|
status=500, content_type=content_type)
|
||||||
else:
|
else:
|
||||||
return HttpResponse(json.dumps({"error": _('Please add folder permission first')}),
|
return HttpResponse(json.dumps({"error": _('Please add folder permission first')}),
|
||||||
status=400, content_type=content_type)
|
status=400, content_type=content_type)
|
||||||
|
Reference in New Issue
Block a user