NextERP/runbot
5
0
Fork 0
mirror of https://github.com/odoo/runbot.git synced 2025-03-27 13:25:47 +07:00
Code Issues Actions Packages Projects Releases Wiki Activity

[IMP] runbot_merge: add signed kw support to from_role, use it

Browse Source 
Closes #774
This commit is contained in:
Xavier Morel 2023-06-07 11:14:47 +02:00
parent 485d2d7b55
commit 611f9150ff
2 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
conftest.py
View File

@ -355,7 +355,7 @@ def dummy_addons_path():
'version': '1.0', 'version': '1.0',
}), encoding='utf-8') }), encoding='utf-8')
(mod / 'util.py').write_text("""\ (mod / 'util.py').write_text("""\
def from_role(_): def from_role(*_, **__):
return lambda fn: fn return lambda fn: fn
""", encoding='utf-8') """, encoding='utf-8')

10
runbot_merge/controllers/reviewer_provisioning.py
View File

@ -6,12 +6,12 @@ from odoo.http import Controller, request, route
try: try:
from odoo.addons.saas_worker.util import from_role from odoo.addons.saas_worker.util import from_role
except ImportError: except ImportError:
def from_role(_): def from_role(*_, **__):
return lambda _: None return lambda _: None
_logger = logging.getLogger(__name__) _logger = logging.getLogger(__name__)
class MergebotReviewerProvisioning(Controller): class MergebotReviewerProvisioning(Controller):
@from_role('accounts') @from_role('accounts', signed=True)
@route('/runbot_merge/users', type='json', auth='public') @route('/runbot_merge/users', type='json', auth='public')
def list_users(self): def list_users(self):
env = request.env(su=True) env = request.env(su=True)
@ -23,7 +23,7 @@ class MergebotReviewerProvisioning(Controller):
if u.github_login if u.github_login
] ]
@from_role('accounts') @from_role('accounts', signed=True)
@route('/runbot_merge/provision', type='json', auth='public') @route('/runbot_merge/provision', type='json', auth='public')
def provision_user(self, users): def provision_user(self, users):
_logger.info('Provisioning %s users: %s.', len(users), ', '.join(map( _logger.info('Provisioning %s users: %s.', len(users), ', '.join(map(
@ -107,7 +107,7 @@ class MergebotReviewerProvisioning(Controller):
_logger.info("Provisioning: created %d updated %d.", created, updated) _logger.info("Provisioning: created %d updated %d.", created, updated)
return [created, updated] return [created, updated]
@from_role('accounts') @from_role('accounts', signed=True)
@route(['/runbot_merge/get_reviewers'], type='json', auth='public') @route(['/runbot_merge/get_reviewers'], type='json', auth='public')
def fetch_reviewers(self, **kwargs): def fetch_reviewers(self, **kwargs):
reviewers = request.env['res.partner.review'].sudo().search([ reviewers = request.env['res.partner.review'].sudo().search([
@ -115,7 +115,7 @@ class MergebotReviewerProvisioning(Controller):
]).mapped('partner_id.github_login') ]).mapped('partner_id.github_login')
return reviewers return reviewers
@from_role('accounts') @from_role('accounts', signed=True)
@route(['/runbot_merge/remove_reviewers'], type='json', auth='public', methods=['POST']) @route(['/runbot_merge/remove_reviewers'], type='json', auth='public', methods=['POST'])
def update_reviewers(self, github_logins, **kwargs): def update_reviewers(self, github_logins, **kwargs):
partners = request.env['res.partner'].sudo().search([('github_login', 'in', github_logins)]) partners = request.env['res.partner'].sudo().search([('github_login', 'in', github_logins)])