The is a simple package with a single shell script. It is lintian clean,
has no dependencies outside of main, has no bug reports, and meets the
UbuntuMainInclusionRequirements. I have performed a brief security audit
and it is mostly fine. I would prefer to see more variables quoted and
noticed a bug in the argument processing. It would also not be difficult
for this script to support SSL. I have taken the liberty of creating a
debdiff which corrects all of these issues. I only lightly tested it, so
people may want to do more testing.

ACK for main inclusion with the attached (or modified) debdiff applied.

** Patch added: "maas-enlist_0.1-0ubuntu2.debdiff"
   
https://bugs.launchpad.net/ubuntu/+source/maas-enlist/+bug/947690/+attachment/2836371/+files/maas-enlist_0.1-0ubuntu2.debdiff

** Changed in: maas-enlist (Ubuntu)
       Status: New => Fix Committed

** Changed in: maas-enlist (Ubuntu)
     Assignee: Jamie Strandboge (jdstrand) => (unassigned)

** Changed in: maas-enlist (Ubuntu)
       Status: Fix Committed => In Progress

** Changed in: maas-enlist (Ubuntu)
     Assignee: (unassigned) => Andres Rodriguez (andreserl)

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/947690

Title:
  [FFe] [MIR] maas-enlist

To manage notifications about this bug go to:
https://bugs.launchpad.net/ubuntu/+source/maas-enlist/+bug/947690/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to