[Sts-sponsors] [Merge] ~igor-brovtsin/maas:lp-1811799 into maas:master

2022-11-21 Thread MAAS Lander
The proposal to merge ~igor-brovtsin/maas:lp-1811799 into maas:master has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433293 -- Your team MAAS Committers is subscribed to branch maas:master. -- Mailing li

[Sts-sponsors] [Merge] ~igor-brovtsin/maas:lp-1811799 into maas:master

2022-11-21 Thread Igor Brovtsin
The proposal to merge ~igor-brovtsin/maas:lp-1811799 into maas:master has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~igor-brovtsin/maas/+git/maas/+merge/433293 -- Your team MAAS Committers is subscribed to branch maas:master. -- Mail

[Sts-sponsors] [Merge] ~igor-brovtsin/maas:lp-1811799 into maas:master

2022-11-21 Thread Igor Brovtsin
The proposal to merge ~igor-brovtsin/maas:lp-1811799 into maas:master has been updated. Commit message changed to: Fix LP #1811799: Normal users can read machine details of owned machines Use `self.read -> get_node_or_404` in `details` handler of nodes API to check permissions For more detail

Re: [Sts-sponsors] [Merge] ~igor-brovtsin/maas:lp-1811799 into maas:master

2022-11-21 Thread Björn Tillenius
Review: Approve Looks good, +1. But please update the commit message to include a reference to the bug: LP #1811799: Normal users can read machine details of owned machines Use `self.read -> get_node_or_404` in `details` handler of nodes API to check permissions -- https://code.launchpad

[Sts-sponsors] [Merge] ~igor-brovtsin/maas:lp-1811799 into maas:master

2022-11-18 Thread Igor Brovtsin
Igor Brovtsin has proposed merging ~igor-brovtsin/maas:lp-1811799 into maas:master. Commit message: Use `self.read -> get_node_or_404` in `details` handler of nodes API to check permissions Requested reviews: MAAS Maintainers (maas-maintainers) Related bugs: Bug #1811799 in MAAS: "[2.5, API