You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@allura.apache.org by gc...@apache.org on 2026/06/16 19:08:18 UTC
(allura) 01/07: [#8607] lock down a bit of the incomplete commit_status endpoint
This is an automated email from the ASF dual-hosted git repository.
asf-gitbox-commits pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/allura.git
commit a5ecc798d28a41699efad3f272a4b4ff90dacb67
Author: Dave Brondsema <da...@brondsema.net>
AuthorDate: Fri Jun 12 10:42:41 2026 -0400
[#8607] lock down a bit of the incomplete commit_status endpoint
---
Allura/allura/controllers/repository.py | 3 +++
1 file changed, 3 insertions(+)
diff --git a/Allura/allura/controllers/repository.py b/Allura/allura/controllers/repository.py
index de6b72ece..dbfad7eb0 100644
--- a/Allura/allura/controllers/repository.py
+++ b/Allura/allura/controllers/repository.py
@@ -373,9 +373,12 @@ def commits(self, rev=None, limit=25, **kw):
]}
@expose('json:')
+ @require_post()
def commit_status(self, rev=None, **kwargs):
+ require_access(c.app.repo, 'admin')
if not g.commit_statuses_enabled:
return {'status': 'disabled', 'message': 'check your config file'}
+ # TODO: if/when this experimental feature is implemented, scope context or app_config field to current repo
params = {x: kwargs.get(x, '').strip() for x in
['state', 'target_url', 'description', 'context']}
params['commit_id'] = rev