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.net/~igor-brovtsin/maas/+git/maas/+merge/433293
Your team MAAS Committers is subscribed to branch maas:master.


-- 
Mailing list: https://launchpad.net/~sts-sponsors
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~sts-sponsors
More help   : https://help.launchpad.net/ListHelp

Reply via email to