launchpad-reviewers team mailing list archive

[Julian Edwards <julian.edwards@xxxxxxxxxxxxx>]

Julian Edwards has proposed merging lp:~julian-edwards/maas/1.2-stop-power-params-leak-bug-1034318 into lp:maas/1.2.

Commit message:
Prevent power parameters being leaked on the API

Requested reviews:
  Julian Edwards (julian-edwards)
Related bugs:
  Bug #1034318 in MAAS: "API calls that return a node leak private data"
  https://bugs.launchpad.net/maas/+bug/1034318

For more details, see:
https://code.launchpad.net/~julian-edwards/maas/1.2-stop-power-params-leak-bug-1034318/+merge/131545
-- 
https://code.launchpad.net/~julian-edwards/maas/1.2-stop-power-params-leak-bug-1034318/+merge/131545
Your team MAAS Maintainers is subscribed to branch lp:maas/1.2.

=== modified file 'src/maasserver/api.py'
--- src/maasserver/api.py	2012-10-11 13:55:44 +0000
+++ src/maasserver/api.py	2012-10-26 08:06:33 +0000
@@ -436,7 +436,6 @@
     'status',
     'netboot',
     'power_type',
-    'power_parameters',
     'tag_names',
     )
 

=== modified file 'src/maasserver/tests/test_api.py'
--- src/maasserver/tests/test_api.py	2012-10-11 13:41:57 +0000
+++ src/maasserver/tests/test_api.py	2012-10-26 08:06:33 +0000
@@ -679,7 +679,6 @@
                 'status',
                 'netboot',
                 'power_type',
-                'power_parameters',
                 'tag_names',
             ],
             list(parsed_result))
@@ -755,7 +754,6 @@
                 'status',
                 'netboot',
                 'power_type',
-                'power_parameters',
                 'resource_uri',
                 'tag_names',
             ],
@@ -896,7 +894,6 @@
                 'status',
                 'netboot',
                 'power_type',
-                'power_parameters',
                 'resource_uri',
                 'tag_names',
             ],