/mandos/trunk

To get this branch, use:
bzr branch http://bzr.recompile.se/loggerhead/mandos/trunk

« back to all changes in this revision

Viewing changes to mandos-ctl

  • Committer: Teddy Hogeborn
  • Date: 2019-03-10 03:33:46 UTC
  • Revision ID: teddy@recompile.se-20190310033346-be20o4gcr3sjkr91
mandos-ctl: Refactor

* mandos-ctl (ValueArgumentMixIn): Remove.
  (PropertyValueCmd): New proper subclass of PropertyCmd; replaces
                      ValueArgumentMixIn.
  (MillisecondsValueArgumentMixIn): Remove.
  (MillisecondsPropertyValueArgumentCmd): New subclass of
            PropertyValueCmd; replaces MillisecondsValueArgumentMixIn.
  (TestValueArgumentPropertyCmd): Rename to "TestPropertyValueCmd".  All users changed.

Show diffs side-by-side

added added

removed removed

Lines of Context:
42
42
import json
43
43
import unittest
44
44
import logging
 
45
import io
 
46
import tempfile
 
47
import contextlib
45
48
 
46
49
import dbus
47
50
 
268
271
    return value
269
272
 
270
273
 
271
 
class TableOfClients(object):
272
 
    tablewords = {
273
 
        "Name": "Name",
274
 
        "Enabled": "Enabled",
275
 
        "Timeout": "Timeout",
276
 
        "LastCheckedOK": "Last Successful Check",
277
 
        "LastApprovalRequest": "Last Approval Request",
278
 
        "Created": "Created",
279
 
        "Interval": "Interval",
280
 
        "Host": "Host",
281
 
        "Fingerprint": "Fingerprint",
282
 
        "KeyID": "Key ID",
283
 
        "CheckerRunning": "Check Is Running",
284
 
        "LastEnabled": "Last Enabled",
285
 
        "ApprovalPending": "Approval Is Pending",
286
 
        "ApprovedByDefault": "Approved By Default",
287
 
        "ApprovalDelay": "Approval Delay",
288
 
        "ApprovalDuration": "Approval Duration",
289
 
        "Checker": "Checker",
290
 
        "ExtendedTimeout": "Extended Timeout",
291
 
        "Expires": "Expires",
292
 
        "LastCheckerStatus": "Last Checker Status",
293
 
    }
294
 
 
295
 
    def __init__(self, clients, keywords, tablewords=None):
296
 
        self.clients = clients
297
 
        self.keywords = keywords
298
 
        if tablewords is not None:
299
 
            self.tablewords = tablewords
300
 
 
301
 
    def __str__(self):
302
 
        return "\n".join(self.rows())
303
 
 
304
 
    if sys.version_info.major == 2:
305
 
        __unicode__ = __str__
306
 
        def __str__(self):
307
 
            return str(self).encode(locale.getpreferredencoding())
308
 
 
309
 
    def rows(self):
310
 
        format_string = self.row_formatting_string()
311
 
        rows = [self.header_line(format_string)]
312
 
        rows.extend(self.client_line(client, format_string)
313
 
                    for client in self.clients)
314
 
        return rows
315
 
 
316
 
    def row_formatting_string(self):
317
 
        "Format string used to format table rows"
318
 
        return " ".join("{{{key}:{width}}}".format(
319
 
            width=max(len(self.tablewords[key]),
320
 
                      *(len(self.string_from_client(client, key))
321
 
                        for client in self.clients)),
322
 
            key=key)
323
 
                        for key in self.keywords)
324
 
 
325
 
    def string_from_client(self, client, key):
326
 
        return self.valuetostring(client[key], key)
327
 
 
328
 
    @staticmethod
329
 
    def valuetostring(value, keyword):
330
 
        if isinstance(value, dbus.Boolean):
331
 
            return "Yes" if value else "No"
332
 
        if keyword in ("Timeout", "Interval", "ApprovalDelay",
333
 
                       "ApprovalDuration", "ExtendedTimeout"):
334
 
            return milliseconds_to_string(value)
335
 
        return str(value)
336
 
 
337
 
    def header_line(self, format_string):
338
 
        return format_string.format(**self.tablewords)
339
 
 
340
 
    def client_line(self, client, format_string):
341
 
        return format_string.format(
342
 
            **{key: self.string_from_client(client, key)
343
 
               for key in self.keywords})
344
 
 
345
 
 
346
274
## Classes for commands.
347
275
 
348
276
# Abstract classes first
349
277
class Command(object):
350
278
    """Abstract class for commands"""
351
 
    def run(self, clients):
 
279
    def run(self, clients, bus=None, mandos=None):
352
280
        """Normal commands should implement run_on_one_client(), but
353
281
        commands which want to operate on all clients at the same time
354
282
        can override this run() method instead."""
355
 
        for client in clients:
356
 
            self.run_on_one_client(client)
 
283
        self.mandos = mandos
 
284
        for clientpath, properties in clients.items():
 
285
            log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
 
286
                      busname, str(clientpath))
 
287
            client = bus.get_object(busname, clientpath)
 
288
            self.run_on_one_client(client, properties)
357
289
 
358
290
class PrintCmd(Command):
359
291
    """Abstract class for commands printing client details"""
364
296
                    "LastApprovalRequest", "ApprovalDelay",
365
297
                    "ApprovalDuration", "Checker", "ExtendedTimeout",
366
298
                    "Expires", "LastCheckerStatus")
367
 
    def run(self, clients):
368
 
        print(self.output(clients))
 
299
    def run(self, clients, bus=None, mandos=None):
 
300
        print(self.output(clients.values()))
 
301
    def output(self, clients):
 
302
        raise NotImplementedError()
369
303
 
370
304
class PropertyCmd(Command):
371
305
    """Abstract class for Actions for setting one client property"""
372
 
    def run_on_one_client(self, client):
 
306
    def run_on_one_client(self, client, properties):
373
307
        """Set the Client's D-Bus property"""
374
 
        client.Set(client_interface, self.property, self.value_to_set,
 
308
        log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", busname,
 
309
                  client.__dbus_object_path__,
 
310
                  dbus.PROPERTIES_IFACE, client_interface,
 
311
                  self.propname, self.value_to_set
 
312
                  if not isinstance(self.value_to_set, dbus.Boolean)
 
313
                  else bool(self.value_to_set))
 
314
        client.Set(client_interface, self.propname, self.value_to_set,
375
315
                   dbus_interface=dbus.PROPERTIES_IFACE)
 
316
    @property
 
317
    def propname(self):
 
318
        raise NotImplementedError()
376
319
 
377
 
class ValueArgumentMixIn(object):
378
 
    """Mixin class for commands taking a value as argument"""
 
320
class PropertyValueCmd(PropertyCmd):
 
321
    """Abstract class for PropertyCmd recieving a value as argument"""
379
322
    def __init__(self, value):
380
323
        self.value_to_set = value
381
324
 
382
 
class MillisecondsValueArgumentMixIn(ValueArgumentMixIn):
383
 
    """Mixin class for commands taking a value argument as
384
 
    milliseconds."""
 
325
class MillisecondsPropertyValueArgumentCmd(PropertyValueCmd):
 
326
    """Abstract class for PropertyValueCmd taking a value argument as
 
327
a datetime.timedelta() but should store it as milliseconds."""
385
328
    @property
386
329
    def value_to_set(self):
387
330
        return self._vts
388
331
    @value_to_set.setter
389
332
    def value_to_set(self, value):
390
 
        """When setting, convert value to a datetime.timedelta"""
391
 
        self._vts = string_to_delta(value).total_seconds() * 1000
 
333
        """When setting, convert value from a datetime.timedelta"""
 
334
        self._vts = int(round(value.total_seconds() * 1000))
392
335
 
393
336
# Actual (non-abstract) command classes
394
337
 
395
338
class PrintTableCmd(PrintCmd):
396
339
    def __init__(self, verbose=False):
397
340
        self.verbose = verbose
 
341
 
398
342
    def output(self, clients):
 
343
        default_keywords = ("Name", "Enabled", "Timeout", "LastCheckedOK")
 
344
        keywords = default_keywords
399
345
        if self.verbose:
400
346
            keywords = self.all_keywords
401
 
        else:
402
 
            keywords = ("Name", "Enabled", "Timeout", "LastCheckedOK")
403
 
        return str(TableOfClients(clients.values(), keywords))
 
347
        return str(self.TableOfClients(clients, keywords))
 
348
 
 
349
    class TableOfClients(object):
 
350
        tableheaders = {
 
351
            "Name": "Name",
 
352
            "Enabled": "Enabled",
 
353
            "Timeout": "Timeout",
 
354
            "LastCheckedOK": "Last Successful Check",
 
355
            "LastApprovalRequest": "Last Approval Request",
 
356
            "Created": "Created",
 
357
            "Interval": "Interval",
 
358
            "Host": "Host",
 
359
            "Fingerprint": "Fingerprint",
 
360
            "KeyID": "Key ID",
 
361
            "CheckerRunning": "Check Is Running",
 
362
            "LastEnabled": "Last Enabled",
 
363
            "ApprovalPending": "Approval Is Pending",
 
364
            "ApprovedByDefault": "Approved By Default",
 
365
            "ApprovalDelay": "Approval Delay",
 
366
            "ApprovalDuration": "Approval Duration",
 
367
            "Checker": "Checker",
 
368
            "ExtendedTimeout": "Extended Timeout",
 
369
            "Expires": "Expires",
 
370
            "LastCheckerStatus": "Last Checker Status",
 
371
        }
 
372
 
 
373
        def __init__(self, clients, keywords, tableheaders=None):
 
374
            self.clients = clients
 
375
            self.keywords = keywords
 
376
            if tableheaders is not None:
 
377
                self.tableheaders = tableheaders
 
378
 
 
379
        def __str__(self):
 
380
            return "\n".join(self.rows())
 
381
 
 
382
        if sys.version_info.major == 2:
 
383
            __unicode__ = __str__
 
384
            def __str__(self):
 
385
                return str(self).encode(locale.getpreferredencoding())
 
386
 
 
387
        def rows(self):
 
388
            format_string = self.row_formatting_string()
 
389
            rows = [self.header_line(format_string)]
 
390
            rows.extend(self.client_line(client, format_string)
 
391
                        for client in self.clients)
 
392
            return rows
 
393
 
 
394
        def row_formatting_string(self):
 
395
            "Format string used to format table rows"
 
396
            return " ".join("{{{key}:{width}}}".format(
 
397
                width=max(len(self.tableheaders[key]),
 
398
                          *(len(self.string_from_client(client, key))
 
399
                            for client in self.clients)),
 
400
                key=key)
 
401
                            for key in self.keywords)
 
402
 
 
403
        def string_from_client(self, client, key):
 
404
            return self.valuetostring(client[key], key)
 
405
 
 
406
        @staticmethod
 
407
        def valuetostring(value, keyword):
 
408
            if isinstance(value, dbus.Boolean):
 
409
                return "Yes" if value else "No"
 
410
            if keyword in ("Timeout", "Interval", "ApprovalDelay",
 
411
                           "ApprovalDuration", "ExtendedTimeout"):
 
412
                return milliseconds_to_string(value)
 
413
            return str(value)
 
414
 
 
415
        def header_line(self, format_string):
 
416
            return format_string.format(**self.tableheaders)
 
417
 
 
418
        def client_line(self, client, format_string):
 
419
            return format_string.format(
 
420
                **{key: self.string_from_client(client, key)
 
421
                   for key in self.keywords})
 
422
 
 
423
 
404
424
 
405
425
class DumpJSONCmd(PrintCmd):
406
426
    def output(self, clients):
416
436
        return value
417
437
 
418
438
class IsEnabledCmd(Command):
419
 
    def run_on_one_client(self, client):
420
 
        if self.is_enabled(client):
 
439
    def run(self, clients, bus=None, mandos=None):
 
440
        client, properties = next(iter(clients.items()))
 
441
        if self.is_enabled(client, properties):
421
442
            sys.exit(0)
422
443
        sys.exit(1)
423
 
    def is_enabled(self, client):
424
 
        return client.Get(client_interface, "Enabled",
425
 
                          dbus_interface=dbus.PROPERTIES_IFACE)
 
444
    def is_enabled(self, client, properties):
 
445
        return properties["Enabled"]
426
446
 
427
447
class RemoveCmd(Command):
428
 
    def __init__(self, mandos):
429
 
        self.mandos = mandos
430
 
    def run_on_one_client(self, client):
 
448
    def run_on_one_client(self, client, properties):
 
449
        log.debug("D-Bus: %s:%s:%s.RemoveClient(%r)", busname,
 
450
                  server_path, server_interface,
 
451
                  str(client.__dbus_object_path__))
431
452
        self.mandos.RemoveClient(client.__dbus_object_path__)
432
453
 
433
454
class ApproveCmd(Command):
434
 
    def run_on_one_client(self, client):
 
455
    def run_on_one_client(self, client, properties):
 
456
        log.debug("D-Bus: %s:%s:%s.Approve(True)", busname,
 
457
                  client.__dbus_object_path__, client_interface)
435
458
        client.Approve(dbus.Boolean(True),
436
459
                       dbus_interface=client_interface)
437
460
 
438
461
class DenyCmd(Command):
439
 
    def run_on_one_client(self, client):
 
462
    def run_on_one_client(self, client, properties):
 
463
        log.debug("D-Bus: %s:%s:%s.Approve(False)", busname,
 
464
                  client.__dbus_object_path__, client_interface)
440
465
        client.Approve(dbus.Boolean(False),
441
466
                       dbus_interface=client_interface)
442
467
 
443
468
class EnableCmd(PropertyCmd):
444
 
    property = "Enabled"
 
469
    propname = "Enabled"
445
470
    value_to_set = dbus.Boolean(True)
446
471
 
447
472
class DisableCmd(PropertyCmd):
448
 
    property = "Enabled"
 
473
    propname = "Enabled"
449
474
    value_to_set = dbus.Boolean(False)
450
475
 
451
476
class BumpTimeoutCmd(PropertyCmd):
452
 
    property = "LastCheckedOK"
 
477
    propname = "LastCheckedOK"
453
478
    value_to_set = ""
454
479
 
455
480
class StartCheckerCmd(PropertyCmd):
456
 
    property = "CheckerRunning"
 
481
    propname = "CheckerRunning"
457
482
    value_to_set = dbus.Boolean(True)
458
483
 
459
484
class StopCheckerCmd(PropertyCmd):
460
 
    property = "CheckerRunning"
 
485
    propname = "CheckerRunning"
461
486
    value_to_set = dbus.Boolean(False)
462
487
 
463
488
class ApproveByDefaultCmd(PropertyCmd):
464
 
    property = "ApprovedByDefault"
 
489
    propname = "ApprovedByDefault"
465
490
    value_to_set = dbus.Boolean(True)
466
491
 
467
492
class DenyByDefaultCmd(PropertyCmd):
468
 
    property = "ApprovedByDefault"
 
493
    propname = "ApprovedByDefault"
469
494
    value_to_set = dbus.Boolean(False)
470
495
 
471
 
class SetCheckerCmd(PropertyCmd, ValueArgumentMixIn):
472
 
    property = "Checker"
473
 
 
474
 
class SetHostCmd(PropertyCmd, ValueArgumentMixIn):
475
 
    property = "Host"
476
 
 
477
 
class SetSecretCmd(PropertyCmd, ValueArgumentMixIn):
478
 
    property = "Secret"
479
 
 
480
 
class SetTimeoutCmd(PropertyCmd, MillisecondsValueArgumentMixIn):
481
 
    property = "Timeout"
482
 
 
483
 
class SetExtendedTimeoutCmd(PropertyCmd,
484
 
                            MillisecondsValueArgumentMixIn):
485
 
    property = "ExtendedTimeout"
486
 
 
487
 
class SetIntervalCmd(PropertyCmd, MillisecondsValueArgumentMixIn):
488
 
    property = "Interval"
489
 
 
490
 
class SetApprovalDelayCmd(PropertyCmd,
491
 
                          MillisecondsValueArgumentMixIn):
492
 
    property = "ApprovalDelay"
493
 
 
494
 
class SetApprovalDurationCmd(PropertyCmd,
495
 
                             MillisecondsValueArgumentMixIn):
496
 
    property = "ApprovalDuration"
497
 
 
498
 
def has_actions(options):
499
 
    return any((options.enable,
500
 
                options.disable,
501
 
                options.bump_timeout,
502
 
                options.start_checker,
503
 
                options.stop_checker,
504
 
                options.is_enabled,
505
 
                options.remove,
506
 
                options.checker is not None,
507
 
                options.timeout is not None,
508
 
                options.extended_timeout is not None,
509
 
                options.interval is not None,
510
 
                options.approved_by_default is not None,
511
 
                options.approval_delay is not None,
512
 
                options.approval_duration is not None,
513
 
                options.host is not None,
514
 
                options.secret is not None,
515
 
                options.approve,
516
 
                options.deny))
517
 
 
518
 
 
519
 
def main():
520
 
    parser = argparse.ArgumentParser()
 
496
class SetCheckerCmd(PropertyValueCmd):
 
497
    propname = "Checker"
 
498
 
 
499
class SetHostCmd(PropertyValueCmd):
 
500
    propname = "Host"
 
501
 
 
502
class SetSecretCmd(PropertyValueCmd):
 
503
    propname = "Secret"
 
504
    @property
 
505
    def value_to_set(self):
 
506
        return self._vts
 
507
    @value_to_set.setter
 
508
    def value_to_set(self, value):
 
509
        """When setting, read data from supplied file object"""
 
510
        self._vts = value.read()
 
511
        value.close()
 
512
 
 
513
class SetTimeoutCmd(MillisecondsPropertyValueArgumentCmd):
 
514
    propname = "Timeout"
 
515
 
 
516
class SetExtendedTimeoutCmd(MillisecondsPropertyValueArgumentCmd):
 
517
    propname = "ExtendedTimeout"
 
518
 
 
519
class SetIntervalCmd(MillisecondsPropertyValueArgumentCmd):
 
520
    propname = "Interval"
 
521
 
 
522
class SetApprovalDelayCmd(MillisecondsPropertyValueArgumentCmd):
 
523
    propname = "ApprovalDelay"
 
524
 
 
525
class SetApprovalDurationCmd(MillisecondsPropertyValueArgumentCmd):
 
526
    propname = "ApprovalDuration"
 
527
 
 
528
def add_command_line_options(parser):
521
529
    parser.add_argument("--version", action="version",
522
530
                        version="%(prog)s {}".format(version),
523
531
                        help="show version number and exit")
548
556
                        help="Remove client")
549
557
    parser.add_argument("-c", "--checker",
550
558
                        help="Set checker command for client")
551
 
    parser.add_argument("-t", "--timeout",
 
559
    parser.add_argument("-t", "--timeout", type=string_to_delta,
552
560
                        help="Set timeout for client")
553
 
    parser.add_argument("--extended-timeout",
 
561
    parser.add_argument("--extended-timeout", type=string_to_delta,
554
562
                        help="Set extended timeout for client")
555
 
    parser.add_argument("-i", "--interval",
 
563
    parser.add_argument("-i", "--interval", type=string_to_delta,
556
564
                        help="Set checker interval for client")
557
565
    approve_deny_default = parser.add_mutually_exclusive_group()
558
566
    approve_deny_default.add_argument(
563
571
        "--deny-by-default", action="store_false",
564
572
        dest="approved_by_default",
565
573
        help="Set client to be denied by default")
566
 
    parser.add_argument("--approval-delay",
 
574
    parser.add_argument("--approval-delay", type=string_to_delta,
567
575
                        help="Set delay before client approve/deny")
568
 
    parser.add_argument("--approval-duration",
 
576
    parser.add_argument("--approval-duration", type=string_to_delta,
569
577
                        help="Set duration of one client approval")
570
578
    parser.add_argument("-H", "--host", help="Set host for client")
571
579
    parser.add_argument("-s", "--secret",
577
585
        help="Approve any current client request")
578
586
    approve_deny.add_argument("-D", "--deny", action="store_true",
579
587
                              help="Deny any current client request")
 
588
    parser.add_argument("--debug", action="store_true",
 
589
                        help="Debug mode (show D-Bus commands)")
580
590
    parser.add_argument("--check", action="store_true",
581
591
                        help="Run self-test")
582
592
    parser.add_argument("client", nargs="*", help="Client name")
583
 
    options = parser.parse_args()
584
 
 
585
 
    if has_actions(options) and not (options.client or options.all):
586
 
        parser.error("Options require clients names or --all.")
587
 
    if options.verbose and has_actions(options):
588
 
        parser.error("--verbose can only be used alone.")
589
 
    if options.dump_json and (options.verbose
590
 
                              or has_actions(options)):
591
 
        parser.error("--dump-json can only be used alone.")
592
 
    if options.all and not has_actions(options):
593
 
        parser.error("--all requires an action.")
594
 
    if options.is_enabled and len(options.client) > 1:
595
 
            parser.error("--is-enabled requires exactly one client")
596
 
 
597
 
    try:
598
 
        bus = dbus.SystemBus()
599
 
        mandos_dbus_objc = bus.get_object(busname, server_path)
600
 
    except dbus.exceptions.DBusException:
601
 
        log.critical("Could not connect to Mandos server")
602
 
        sys.exit(1)
603
 
 
604
 
    mandos_serv = dbus.Interface(mandos_dbus_objc,
605
 
                                 dbus_interface=server_interface)
606
 
    mandos_serv_object_manager = dbus.Interface(
607
 
        mandos_dbus_objc, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
 
593
 
 
594
 
 
595
def commands_from_options(options):
608
596
 
609
597
    commands = []
610
598
 
618
606
        commands.append(DisableCmd())
619
607
 
620
608
    if options.bump_timeout:
621
 
        commands.append(BumpTimeoutCmd(options.bump_timeout))
 
609
        commands.append(BumpTimeoutCmd())
622
610
 
623
611
    if options.start_checker:
624
612
        commands.append(StartCheckerCmd())
629
617
    if options.is_enabled:
630
618
        commands.append(IsEnabledCmd())
631
619
 
632
 
    if options.remove:
633
 
        commands.append(RemoveCmd(mandos_serv))
634
 
 
635
620
    if options.checker is not None:
636
 
        commands.append(SetCheckerCmd())
 
621
        commands.append(SetCheckerCmd(options.checker))
637
622
 
638
623
    if options.timeout is not None:
639
624
        commands.append(SetTimeoutCmd(options.timeout))
643
628
            SetExtendedTimeoutCmd(options.extended_timeout))
644
629
 
645
630
    if options.interval is not None:
646
 
        command.append(SetIntervalCmd(options.interval))
 
631
        commands.append(SetIntervalCmd(options.interval))
647
632
 
648
633
    if options.approved_by_default is not None:
649
634
        if options.approved_by_default:
650
 
            command.append(ApproveByDefaultCmd())
 
635
            commands.append(ApproveByDefaultCmd())
651
636
        else:
652
 
            command.append(DenyByDefaultCmd())
 
637
            commands.append(DenyByDefaultCmd())
653
638
 
654
639
    if options.approval_delay is not None:
655
 
        command.append(SetApprovalDelayCmd(options.approval_delay))
 
640
        commands.append(SetApprovalDelayCmd(options.approval_delay))
656
641
 
657
642
    if options.approval_duration is not None:
658
 
        command.append(
 
643
        commands.append(
659
644
            SetApprovalDurationCmd(options.approval_duration))
660
645
 
661
646
    if options.host is not None:
662
 
        command.append(SetHostCmd(options.host))
 
647
        commands.append(SetHostCmd(options.host))
663
648
 
664
649
    if options.secret is not None:
665
 
        command.append(SetSecretCmd(options.secret))
 
650
        commands.append(SetSecretCmd(options.secret))
666
651
 
667
652
    if options.approve:
668
653
        commands.append(ApproveCmd())
670
655
    if options.deny:
671
656
        commands.append(DenyCmd())
672
657
 
 
658
    if options.remove:
 
659
        commands.append(RemoveCmd())
 
660
 
673
661
    # If no command option has been given, show table of clients,
674
662
    # optionally verbosely
675
663
    if not commands:
676
664
        commands.append(PrintTableCmd(verbose=options.verbose))
677
665
 
678
 
    # block stderr since dbus library prints to stderr
679
 
    null = os.open(os.path.devnull, os.O_RDWR)
680
 
    stderrcopy = os.dup(sys.stderr.fileno())
681
 
    os.dup2(null, sys.stderr.fileno())
682
 
    os.close(null)
683
 
    try:
684
 
        try:
685
 
            mandos_clients = {path: ifs_and_props[client_interface]
686
 
                              for path, ifs_and_props in
687
 
                              mandos_serv_object_manager
688
 
                              .GetManagedObjects().items()
689
 
                              if client_interface in ifs_and_props}
690
 
        finally:
691
 
            # restore stderr
692
 
            os.dup2(stderrcopy, sys.stderr.fileno())
693
 
            os.close(stderrcopy)
 
666
    return commands
 
667
 
 
668
 
 
669
def check_option_syntax(parser, options):
 
670
    """Apply additional restrictions on options, not expressible in
 
671
argparse"""
 
672
 
 
673
    def has_actions(options):
 
674
        return any((options.enable,
 
675
                    options.disable,
 
676
                    options.bump_timeout,
 
677
                    options.start_checker,
 
678
                    options.stop_checker,
 
679
                    options.is_enabled,
 
680
                    options.remove,
 
681
                    options.checker is not None,
 
682
                    options.timeout is not None,
 
683
                    options.extended_timeout is not None,
 
684
                    options.interval is not None,
 
685
                    options.approved_by_default is not None,
 
686
                    options.approval_delay is not None,
 
687
                    options.approval_duration is not None,
 
688
                    options.host is not None,
 
689
                    options.secret is not None,
 
690
                    options.approve,
 
691
                    options.deny))
 
692
 
 
693
    if has_actions(options) and not (options.client or options.all):
 
694
        parser.error("Options require clients names or --all.")
 
695
    if options.verbose and has_actions(options):
 
696
        parser.error("--verbose can only be used alone.")
 
697
    if options.dump_json and (options.verbose
 
698
                              or has_actions(options)):
 
699
        parser.error("--dump-json can only be used alone.")
 
700
    if options.all and not has_actions(options):
 
701
        parser.error("--all requires an action.")
 
702
    if options.is_enabled and len(options.client) > 1:
 
703
        parser.error("--is-enabled requires exactly one client")
 
704
    if options.remove:
 
705
        options.remove = False
 
706
        if has_actions(options) and not options.deny:
 
707
            parser.error("--remove can only be combined with --deny")
 
708
        options.remove = True
 
709
 
 
710
 
 
711
def main():
 
712
    parser = argparse.ArgumentParser()
 
713
 
 
714
    add_command_line_options(parser)
 
715
 
 
716
    options = parser.parse_args()
 
717
 
 
718
    check_option_syntax(parser, options)
 
719
 
 
720
    clientnames = options.client
 
721
 
 
722
    if options.debug:
 
723
        log.setLevel(logging.DEBUG)
 
724
 
 
725
    try:
 
726
        bus = dbus.SystemBus()
 
727
        log.debug("D-Bus: Connect to: (busname=%r, path=%r)", busname,
 
728
                  server_path)
 
729
        mandos_dbus_objc = bus.get_object(busname, server_path)
 
730
    except dbus.exceptions.DBusException:
 
731
        log.critical("Could not connect to Mandos server")
 
732
        sys.exit(1)
 
733
 
 
734
    mandos_serv = dbus.Interface(mandos_dbus_objc,
 
735
                                 dbus_interface=server_interface)
 
736
    mandos_serv_object_manager = dbus.Interface(
 
737
        mandos_dbus_objc, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
 
738
 
 
739
    # Filter out log message from dbus module
 
740
    dbus_logger = logging.getLogger("dbus.proxies")
 
741
    class NullFilter(logging.Filter):
 
742
        def filter(self, record):
 
743
            return False
 
744
    dbus_filter = NullFilter()
 
745
    try:
 
746
        dbus_logger.addFilter(dbus_filter)
 
747
        log.debug("D-Bus: %s:%s:%s.GetManagedObjects()", busname,
 
748
                  server_path, dbus.OBJECT_MANAGER_IFACE)
 
749
        mandos_clients = {path: ifs_and_props[client_interface]
 
750
                          for path, ifs_and_props in
 
751
                          mandos_serv_object_manager
 
752
                          .GetManagedObjects().items()
 
753
                          if client_interface in ifs_and_props}
694
754
    except dbus.exceptions.DBusException as e:
695
755
        log.critical("Failed to access Mandos server through D-Bus:"
696
756
                     "\n%s", e)
697
757
        sys.exit(1)
 
758
    finally:
 
759
        # restore dbus logger
 
760
        dbus_logger.removeFilter(dbus_filter)
698
761
 
699
762
    # Compile dict of (clients: properties) to process
700
763
    clients = {}
701
764
 
702
 
    if options.all or not options.client:
703
 
        clients = {bus.get_object(busname, path): properties
704
 
                   for path, properties in mandos_clients.items()}
 
765
    if not clientnames:
 
766
        clients = {objpath: properties
 
767
                   for objpath, properties in mandos_clients.items()}
705
768
    else:
706
 
        for name in options.client:
707
 
            for path, client in mandos_clients.items():
708
 
                if client["Name"] == name:
709
 
                    client_objc = bus.get_object(busname, path)
710
 
                    clients[client_objc] = client
 
769
        for name in clientnames:
 
770
            for objpath, properties in mandos_clients.items():
 
771
                if properties["Name"] == name:
 
772
                    clients[objpath] = properties
711
773
                    break
712
774
            else:
713
775
                log.critical("Client not found on server: %r", name)
714
776
                sys.exit(1)
715
777
 
716
778
    # Run all commands on clients
 
779
    commands = commands_from_options(options)
717
780
    for command in commands:
718
 
        command.run(clients)
 
781
        command.run(clients, bus, mandos_serv)
719
782
 
720
783
 
721
784
class Test_milliseconds_to_string(unittest.TestCase):
733
796
 
734
797
class Test_string_to_delta(unittest.TestCase):
735
798
    def test_handles_basic_rfc3339(self):
 
799
        self.assertEqual(string_to_delta("PT0S"),
 
800
                         datetime.timedelta())
 
801
        self.assertEqual(string_to_delta("P0D"),
 
802
                         datetime.timedelta())
 
803
        self.assertEqual(string_to_delta("PT1S"),
 
804
                         datetime.timedelta(0, 1))
736
805
        self.assertEqual(string_to_delta("PT2H"),
737
806
                         datetime.timedelta(0, 7200))
738
807
    def test_falls_back_to_pre_1_6_1_with_warning(self):
741
810
            with self.assertLogs(log, logging.WARNING):
742
811
                value = string_to_delta("2h")
743
812
        else:
744
 
            value = string_to_delta("2h")
 
813
            class WarningFilter(logging.Filter):
 
814
                """Don't show, but record the presence of, warnings"""
 
815
                def filter(self, record):
 
816
                    is_warning = record.levelno >= logging.WARNING
 
817
                    self.found = is_warning or getattr(self, "found",
 
818
                                                       False)
 
819
                    return not is_warning
 
820
            warning_filter = WarningFilter()
 
821
            log.addFilter(warning_filter)
 
822
            try:
 
823
                value = string_to_delta("2h")
 
824
            finally:
 
825
                log.removeFilter(warning_filter)
 
826
            self.assertTrue(getattr(warning_filter, "found", False))
745
827
        self.assertEqual(value, datetime.timedelta(0, 7200))
746
828
 
747
 
class Test_TableOfClients(unittest.TestCase):
748
 
    def setUp(self):
749
 
        self.tablewords = {
750
 
            "Attr1": "X",
751
 
            "AttrTwo": "Yy",
752
 
            "AttrThree": "Zzz",
753
 
            "Bool": "A D-BUS Boolean",
754
 
            "NonDbusBoolean": "A Non-D-BUS Boolean",
755
 
            "Integer": "An Integer",
756
 
            "Timeout": "Timedelta 1",
757
 
            "Interval": "Timedelta 2",
758
 
            "ApprovalDelay": "Timedelta 3",
759
 
            "ApprovalDuration": "Timedelta 4",
760
 
            "ExtendedTimeout": "Timedelta 5",
761
 
            "String": "A String",
 
829
 
 
830
class TestCmd(unittest.TestCase):
 
831
    """Abstract class for tests of command classes"""
 
832
    def setUp(self):
 
833
        testcase = self
 
834
        class MockClient(object):
 
835
            def __init__(self, name, **attributes):
 
836
                self.__dbus_object_path__ = "/clients/{}".format(name)
 
837
                self.attributes = attributes
 
838
                self.attributes["Name"] = name
 
839
                self.calls = []
 
840
            def Set(self, interface, propname, value, dbus_interface):
 
841
                testcase.assertEqual(interface, client_interface)
 
842
                testcase.assertEqual(dbus_interface,
 
843
                                     dbus.PROPERTIES_IFACE)
 
844
                self.attributes[propname] = value
 
845
            def Get(self, interface, propname, dbus_interface):
 
846
                testcase.assertEqual(interface, client_interface)
 
847
                testcase.assertEqual(dbus_interface,
 
848
                                     dbus.PROPERTIES_IFACE)
 
849
                return self.attributes[propname]
 
850
            def Approve(self, approve, dbus_interface):
 
851
                testcase.assertEqual(dbus_interface, client_interface)
 
852
                self.calls.append(("Approve", (approve,
 
853
                                               dbus_interface)))
 
854
        self.client = MockClient(
 
855
            "foo",
 
856
            KeyID=("92ed150794387c03ce684574b1139a65"
 
857
                   "94a34f895daaaf09fd8ea90a27cddb12"),
 
858
            Secret=b"secret",
 
859
            Host="foo.example.org",
 
860
            Enabled=dbus.Boolean(True),
 
861
            Timeout=300000,
 
862
            LastCheckedOK="2019-02-03T00:00:00",
 
863
            Created="2019-01-02T00:00:00",
 
864
            Interval=120000,
 
865
            Fingerprint=("778827225BA7DE539C5A"
 
866
                         "7CFA59CFF7CDBD9A5920"),
 
867
            CheckerRunning=dbus.Boolean(False),
 
868
            LastEnabled="2019-01-03T00:00:00",
 
869
            ApprovalPending=dbus.Boolean(False),
 
870
            ApprovedByDefault=dbus.Boolean(True),
 
871
            LastApprovalRequest="",
 
872
            ApprovalDelay=0,
 
873
            ApprovalDuration=1000,
 
874
            Checker="fping -q -- %(host)s",
 
875
            ExtendedTimeout=900000,
 
876
            Expires="2019-02-04T00:00:00",
 
877
            LastCheckerStatus=0)
 
878
        self.other_client = MockClient(
 
879
            "barbar",
 
880
            KeyID=("0558568eedd67d622f5c83b35a115f79"
 
881
                   "6ab612cff5ad227247e46c2b020f441c"),
 
882
            Secret=b"secretbar",
 
883
            Host="192.0.2.3",
 
884
            Enabled=dbus.Boolean(True),
 
885
            Timeout=300000,
 
886
            LastCheckedOK="2019-02-04T00:00:00",
 
887
            Created="2019-01-03T00:00:00",
 
888
            Interval=120000,
 
889
            Fingerprint=("3E393AEAEFB84C7E89E2"
 
890
                         "F547B3A107558FCA3A27"),
 
891
            CheckerRunning=dbus.Boolean(True),
 
892
            LastEnabled="2019-01-04T00:00:00",
 
893
            ApprovalPending=dbus.Boolean(False),
 
894
            ApprovedByDefault=dbus.Boolean(False),
 
895
            LastApprovalRequest="2019-01-03T00:00:00",
 
896
            ApprovalDelay=30000,
 
897
            ApprovalDuration=1000,
 
898
            Checker=":",
 
899
            ExtendedTimeout=900000,
 
900
            Expires="2019-02-05T00:00:00",
 
901
            LastCheckerStatus=-2)
 
902
        self.clients =  collections.OrderedDict(
 
903
            [
 
904
                ("/clients/foo", self.client.attributes),
 
905
                ("/clients/barbar", self.other_client.attributes),
 
906
            ])
 
907
        self.one_client = {"/clients/foo": self.client.attributes}
 
908
    @property
 
909
    def bus(self):
 
910
        class Bus(object):
 
911
            @staticmethod
 
912
            def get_object(client_bus_name, path):
 
913
                self.assertEqual(client_bus_name, busname)
 
914
                return {
 
915
                    "/clients/foo": self.client,
 
916
                    "/clients/barbar": self.other_client,
 
917
                }[path]
 
918
        return Bus()
 
919
 
 
920
class TestPrintTableCmd(TestCmd):
 
921
    def test_normal(self):
 
922
        output = PrintTableCmd().output(self.clients.values())
 
923
        expected_output = """
 
924
Name   Enabled Timeout  Last Successful Check
 
925
foo    Yes     00:05:00 2019-02-03T00:00:00  
 
926
barbar Yes     00:05:00 2019-02-04T00:00:00  
 
927
"""[1:-1]
 
928
        self.assertEqual(output, expected_output)
 
929
    def test_verbose(self):
 
930
        output = PrintTableCmd(verbose=True).output(
 
931
            self.clients.values())
 
932
        expected_output = """
 
933
Name   Enabled Timeout  Last Successful Check Created             Interval Host            Key ID                                                           Fingerprint                              Check Is Running Last Enabled        Approval Is Pending Approved By Default Last Approval Request Approval Delay Approval Duration Checker              Extended Timeout Expires             Last Checker Status
 
934
foo    Yes     00:05:00 2019-02-03T00:00:00   2019-01-02T00:00:00 00:02:00 foo.example.org 92ed150794387c03ce684574b1139a6594a34f895daaaf09fd8ea90a27cddb12 778827225BA7DE539C5A7CFA59CFF7CDBD9A5920 No               2019-01-03T00:00:00 No                  Yes                                       00:00:00       00:00:01          fping -q -- %(host)s 00:15:00         2019-02-04T00:00:00 0                  
 
935
barbar Yes     00:05:00 2019-02-04T00:00:00   2019-01-03T00:00:00 00:02:00 192.0.2.3       0558568eedd67d622f5c83b35a115f796ab612cff5ad227247e46c2b020f441c 3E393AEAEFB84C7E89E2F547B3A107558FCA3A27 Yes              2019-01-04T00:00:00 No                  No                  2019-01-03T00:00:00   00:00:30       00:00:01          :                    00:15:00         2019-02-05T00:00:00 -2                 
 
936
"""[1:-1]
 
937
        self.assertEqual(output, expected_output)
 
938
    def test_one_client(self):
 
939
        output = PrintTableCmd().output(self.one_client.values())
 
940
        expected_output = """
 
941
Name Enabled Timeout  Last Successful Check
 
942
foo  Yes     00:05:00 2019-02-03T00:00:00  
 
943
"""[1:-1]
 
944
        self.assertEqual(output, expected_output)
 
945
 
 
946
class TestDumpJSONCmd(TestCmd):
 
947
    def setUp(self):
 
948
        self.expected_json = {
 
949
            "foo": {
 
950
                "Name": "foo",
 
951
                "KeyID": ("92ed150794387c03ce684574b1139a65"
 
952
                          "94a34f895daaaf09fd8ea90a27cddb12"),
 
953
                "Host": "foo.example.org",
 
954
                "Enabled": True,
 
955
                "Timeout": 300000,
 
956
                "LastCheckedOK": "2019-02-03T00:00:00",
 
957
                "Created": "2019-01-02T00:00:00",
 
958
                "Interval": 120000,
 
959
                "Fingerprint": ("778827225BA7DE539C5A"
 
960
                                "7CFA59CFF7CDBD9A5920"),
 
961
                "CheckerRunning": False,
 
962
                "LastEnabled": "2019-01-03T00:00:00",
 
963
                "ApprovalPending": False,
 
964
                "ApprovedByDefault": True,
 
965
                "LastApprovalRequest": "",
 
966
                "ApprovalDelay": 0,
 
967
                "ApprovalDuration": 1000,
 
968
                "Checker": "fping -q -- %(host)s",
 
969
                "ExtendedTimeout": 900000,
 
970
                "Expires": "2019-02-04T00:00:00",
 
971
                "LastCheckerStatus": 0,
 
972
            },
 
973
            "barbar": {
 
974
                "Name": "barbar",
 
975
                "KeyID": ("0558568eedd67d622f5c83b35a115f79"
 
976
                          "6ab612cff5ad227247e46c2b020f441c"),
 
977
                "Host": "192.0.2.3",
 
978
                "Enabled": True,
 
979
                "Timeout": 300000,
 
980
                "LastCheckedOK": "2019-02-04T00:00:00",
 
981
                "Created": "2019-01-03T00:00:00",
 
982
                "Interval": 120000,
 
983
                "Fingerprint": ("3E393AEAEFB84C7E89E2"
 
984
                                "F547B3A107558FCA3A27"),
 
985
                "CheckerRunning": True,
 
986
                "LastEnabled": "2019-01-04T00:00:00",
 
987
                "ApprovalPending": False,
 
988
                "ApprovedByDefault": False,
 
989
                "LastApprovalRequest": "2019-01-03T00:00:00",
 
990
                "ApprovalDelay": 30000,
 
991
                "ApprovalDuration": 1000,
 
992
                "Checker": ":",
 
993
                "ExtendedTimeout": 900000,
 
994
                "Expires": "2019-02-05T00:00:00",
 
995
                "LastCheckerStatus": -2,
 
996
            },
762
997
        }
763
 
        self.keywords = ["Attr1", "AttrTwo"]
764
 
        self.clients = [
765
 
            {
766
 
                "Attr1": "x1",
767
 
                "AttrTwo": "y1",
768
 
                "AttrThree": "z1",
769
 
                "Bool": dbus.Boolean(False),
770
 
                "NonDbusBoolean": False,
771
 
                "Integer": 0,
772
 
                "Timeout": 0,
773
 
                "Interval": 1000,
774
 
                "ApprovalDelay": 2000,
775
 
                "ApprovalDuration": 3000,
776
 
                "ExtendedTimeout": 4000,
777
 
                "String": "",
778
 
            },
779
 
            {
780
 
                "Attr1": "x2",
781
 
                "AttrTwo": "y2",
782
 
                "AttrThree": "z2",
783
 
                "Bool": dbus.Boolean(True),
784
 
                "NonDbusBoolean": True,
785
 
                "Integer": 1,
786
 
                "Timeout": 93785000,
787
 
                "Interval": 93786000,
788
 
                "ApprovalDelay": 93787000,
789
 
                "ApprovalDuration": 93788000,
790
 
                "ExtendedTimeout": 93789000,
791
 
                "String": "A huge string which will not fit," * 10,
792
 
            },
793
 
        ]
794
 
    def test_short_header(self):
795
 
        text = str(TableOfClients(self.clients, self.keywords,
796
 
                                  self.tablewords))
797
 
        expected_text = """
798
 
X  Yy
799
 
x1 y1
800
 
x2 y2
801
 
"""[1:-1]
802
 
        self.assertEqual(text, expected_text)
803
 
    def test_booleans(self):
804
 
        keywords = ["Bool", "NonDbusBoolean"]
805
 
        text = str(TableOfClients(self.clients, keywords,
806
 
                                  self.tablewords))
807
 
        expected_text = """
808
 
A D-BUS Boolean A Non-D-BUS Boolean
809
 
No              False              
810
 
Yes             True               
811
 
"""[1:-1]
812
 
        self.assertEqual(text, expected_text)
813
 
    def test_milliseconds_detection(self):
814
 
        keywords = ["Integer", "Timeout", "Interval", "ApprovalDelay",
815
 
                    "ApprovalDuration", "ExtendedTimeout"]
816
 
        text = str(TableOfClients(self.clients, keywords,
817
 
                                  self.tablewords))
818
 
        expected_text = """
819
 
An Integer Timedelta 1 Timedelta 2 Timedelta 3 Timedelta 4 Timedelta 5
820
 
0          00:00:00    00:00:01    00:00:02    00:00:03    00:00:04   
821
 
1          1T02:03:05  1T02:03:06  1T02:03:07  1T02:03:08  1T02:03:09 
822
 
"""[1:-1]
823
 
        self.assertEqual(text, expected_text)
824
 
    def test_empty_and_long_string_values(self):
825
 
        keywords = ["String"]
826
 
        text = str(TableOfClients(self.clients, keywords,
827
 
                                  self.tablewords))
828
 
        expected_text = """
829
 
A String                                                                                                                                                                                                                                                                                                                                  
830
 
                                                                                                                                                                                                                                                                                                                                          
831
 
A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,A huge string which will not fit,
832
 
"""[1:-1]
833
 
        self.assertEqual(text, expected_text)
 
998
        return super(TestDumpJSONCmd, self).setUp()
 
999
    def test_normal(self):
 
1000
        json_data = json.loads(DumpJSONCmd().output(self.clients))
 
1001
        self.assertDictEqual(json_data, self.expected_json)
 
1002
    def test_one_client(self):
 
1003
        clients = self.one_client
 
1004
        json_data = json.loads(DumpJSONCmd().output(clients))
 
1005
        expected_json = {"foo": self.expected_json["foo"]}
 
1006
        self.assertDictEqual(json_data, expected_json)
 
1007
 
 
1008
class TestIsEnabledCmd(TestCmd):
 
1009
    def test_is_enabled(self):
 
1010
        self.assertTrue(all(IsEnabledCmd().is_enabled(client, properties)
 
1011
                            for client, properties in self.clients.items()))
 
1012
    def test_is_enabled_run_exits_successfully(self):
 
1013
        with self.assertRaises(SystemExit) as e:
 
1014
            IsEnabledCmd().run(self.one_client)
 
1015
        if e.exception.code is not None:
 
1016
            self.assertEqual(e.exception.code, 0)
 
1017
        else:
 
1018
            self.assertIsNone(e.exception.code)
 
1019
    def test_is_enabled_run_exits_with_failure(self):
 
1020
        self.client.attributes["Enabled"] = dbus.Boolean(False)
 
1021
        with self.assertRaises(SystemExit) as e:
 
1022
            IsEnabledCmd().run(self.one_client)
 
1023
        if isinstance(e.exception.code, int):
 
1024
            self.assertNotEqual(e.exception.code, 0)
 
1025
        else:
 
1026
            self.assertIsNotNone(e.exception.code)
 
1027
 
 
1028
class TestRemoveCmd(TestCmd):
 
1029
    def test_remove(self):
 
1030
        class MockMandos(object):
 
1031
            def __init__(self):
 
1032
                self.calls = []
 
1033
            def RemoveClient(self, dbus_path):
 
1034
                self.calls.append(("RemoveClient", (dbus_path,)))
 
1035
        mandos = MockMandos()
 
1036
        super(TestRemoveCmd, self).setUp()
 
1037
        RemoveCmd().run(self.clients, self.bus, mandos)
 
1038
        self.assertEqual(len(mandos.calls), 2)
 
1039
        for clientpath in self.clients:
 
1040
            self.assertIn(("RemoveClient", (clientpath,)),
 
1041
                          mandos.calls)
 
1042
 
 
1043
class TestApproveCmd(TestCmd):
 
1044
    def test_approve(self):
 
1045
        ApproveCmd().run(self.clients, self.bus)
 
1046
        for clientpath in self.clients:
 
1047
            client = self.bus.get_object(busname, clientpath)
 
1048
            self.assertIn(("Approve", (True, client_interface)),
 
1049
                          client.calls)
 
1050
 
 
1051
class TestDenyCmd(TestCmd):
 
1052
    def test_deny(self):
 
1053
        DenyCmd().run(self.clients, self.bus)
 
1054
        for clientpath in self.clients:
 
1055
            client = self.bus.get_object(busname, clientpath)
 
1056
            self.assertIn(("Approve", (False, client_interface)),
 
1057
                          client.calls)
 
1058
 
 
1059
class TestEnableCmd(TestCmd):
 
1060
    def test_enable(self):
 
1061
        for clientpath in self.clients:
 
1062
            client = self.bus.get_object(busname, clientpath)
 
1063
            client.attributes["Enabled"] = False
 
1064
 
 
1065
        EnableCmd().run(self.clients, self.bus)
 
1066
 
 
1067
        for clientpath in self.clients:
 
1068
            client = self.bus.get_object(busname, clientpath)
 
1069
            self.assertTrue(client.attributes["Enabled"])
 
1070
 
 
1071
class TestDisableCmd(TestCmd):
 
1072
    def test_disable(self):
 
1073
        DisableCmd().run(self.clients, self.bus)
 
1074
        for clientpath in self.clients:
 
1075
            client = self.bus.get_object(busname, clientpath)
 
1076
            self.assertFalse(client.attributes["Enabled"])
 
1077
 
 
1078
class Unique(object):
 
1079
    """Class for objects which exist only to be unique objects, since
 
1080
unittest.mock.sentinel only exists in Python 3.3"""
 
1081
 
 
1082
class TestPropertyCmd(TestCmd):
 
1083
    """Abstract class for tests of PropertyCmd classes"""
 
1084
    def runTest(self):
 
1085
        if not hasattr(self, "command"):
 
1086
            return
 
1087
        values_to_get = getattr(self, "values_to_get",
 
1088
                                self.values_to_set)
 
1089
        for value_to_set, value_to_get in zip(self.values_to_set,
 
1090
                                              values_to_get):
 
1091
            for clientpath in self.clients:
 
1092
                client = self.bus.get_object(busname, clientpath)
 
1093
                old_value = client.attributes[self.propname]
 
1094
                self.assertNotIsInstance(old_value, Unique)
 
1095
                client.attributes[self.propname] = Unique()
 
1096
            self.run_command(value_to_set, self.clients)
 
1097
            for clientpath in self.clients:
 
1098
                client = self.bus.get_object(busname, clientpath)
 
1099
                value = client.attributes[self.propname]
 
1100
                self.assertNotIsInstance(value, Unique)
 
1101
                self.assertEqual(value, value_to_get)
 
1102
    def run_command(self, value, clients):
 
1103
        self.command().run(clients, self.bus)
 
1104
 
 
1105
class TestBumpTimeoutCmd(TestPropertyCmd):
 
1106
    command = BumpTimeoutCmd
 
1107
    propname = "LastCheckedOK"
 
1108
    values_to_set = [""]
 
1109
 
 
1110
class TestStartCheckerCmd(TestPropertyCmd):
 
1111
    command = StartCheckerCmd
 
1112
    propname = "CheckerRunning"
 
1113
    values_to_set = [dbus.Boolean(True)]
 
1114
 
 
1115
class TestStopCheckerCmd(TestPropertyCmd):
 
1116
    command = StopCheckerCmd
 
1117
    propname = "CheckerRunning"
 
1118
    values_to_set = [dbus.Boolean(False)]
 
1119
 
 
1120
class TestApproveByDefaultCmd(TestPropertyCmd):
 
1121
    command = ApproveByDefaultCmd
 
1122
    propname = "ApprovedByDefault"
 
1123
    values_to_set = [dbus.Boolean(True)]
 
1124
 
 
1125
class TestDenyByDefaultCmd(TestPropertyCmd):
 
1126
    command = DenyByDefaultCmd
 
1127
    propname = "ApprovedByDefault"
 
1128
    values_to_set = [dbus.Boolean(False)]
 
1129
 
 
1130
class TestPropertyValueCmd(TestPropertyCmd):
 
1131
    """Abstract class for tests of PropertyValueCmd classes"""
 
1132
    def runTest(self):
 
1133
        if type(self) is TestPropertyValueCmd:
 
1134
            return
 
1135
        return super(TestPropertyValueCmd, self).runTest()
 
1136
    def run_command(self, value, clients):
 
1137
        self.command(value).run(clients, self.bus)
 
1138
 
 
1139
class TestSetCheckerCmd(TestPropertyValueCmd):
 
1140
    command = SetCheckerCmd
 
1141
    propname = "Checker"
 
1142
    values_to_set = ["", ":", "fping -q -- %s"]
 
1143
 
 
1144
class TestSetHostCmd(TestPropertyValueCmd):
 
1145
    command = SetHostCmd
 
1146
    propname = "Host"
 
1147
    values_to_set = ["192.0.2.3", "foo.example.org"]
 
1148
 
 
1149
class TestSetSecretCmd(TestPropertyValueCmd):
 
1150
    command = SetSecretCmd
 
1151
    propname = "Secret"
 
1152
    values_to_set = [io.BytesIO(b""),
 
1153
                     io.BytesIO(b"secret\0xyzzy\nbar")]
 
1154
    values_to_get = [b"", b"secret\0xyzzy\nbar"]
 
1155
 
 
1156
class TestSetTimeoutCmd(TestPropertyValueCmd):
 
1157
    command = SetTimeoutCmd
 
1158
    propname = "Timeout"
 
1159
    values_to_set = [datetime.timedelta(),
 
1160
                     datetime.timedelta(minutes=5),
 
1161
                     datetime.timedelta(seconds=1),
 
1162
                     datetime.timedelta(weeks=1),
 
1163
                     datetime.timedelta(weeks=52)]
 
1164
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1165
 
 
1166
class TestSetExtendedTimeoutCmd(TestPropertyValueCmd):
 
1167
    command = SetExtendedTimeoutCmd
 
1168
    propname = "ExtendedTimeout"
 
1169
    values_to_set = [datetime.timedelta(),
 
1170
                     datetime.timedelta(minutes=5),
 
1171
                     datetime.timedelta(seconds=1),
 
1172
                     datetime.timedelta(weeks=1),
 
1173
                     datetime.timedelta(weeks=52)]
 
1174
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1175
 
 
1176
class TestSetIntervalCmd(TestPropertyValueCmd):
 
1177
    command = SetIntervalCmd
 
1178
    propname = "Interval"
 
1179
    values_to_set = [datetime.timedelta(),
 
1180
                     datetime.timedelta(minutes=5),
 
1181
                     datetime.timedelta(seconds=1),
 
1182
                     datetime.timedelta(weeks=1),
 
1183
                     datetime.timedelta(weeks=52)]
 
1184
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1185
 
 
1186
class TestSetApprovalDelayCmd(TestPropertyValueCmd):
 
1187
    command = SetApprovalDelayCmd
 
1188
    propname = "ApprovalDelay"
 
1189
    values_to_set = [datetime.timedelta(),
 
1190
                     datetime.timedelta(minutes=5),
 
1191
                     datetime.timedelta(seconds=1),
 
1192
                     datetime.timedelta(weeks=1),
 
1193
                     datetime.timedelta(weeks=52)]
 
1194
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1195
 
 
1196
class TestSetApprovalDurationCmd(TestPropertyValueCmd):
 
1197
    command = SetApprovalDurationCmd
 
1198
    propname = "ApprovalDuration"
 
1199
    values_to_set = [datetime.timedelta(),
 
1200
                     datetime.timedelta(minutes=5),
 
1201
                     datetime.timedelta(seconds=1),
 
1202
                     datetime.timedelta(weeks=1),
 
1203
                     datetime.timedelta(weeks=52)]
 
1204
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1205
 
 
1206
class Test_command_from_options(unittest.TestCase):
 
1207
    def setUp(self):
 
1208
        self.parser = argparse.ArgumentParser()
 
1209
        add_command_line_options(self.parser)
 
1210
    def assert_command_from_args(self, args, command_cls, **cmd_attrs):
 
1211
        """Assert that parsing ARGS should result in an instance of
 
1212
COMMAND_CLS with (optionally) all supplied attributes (CMD_ATTRS)."""
 
1213
        options = self.parser.parse_args(args)
 
1214
        check_option_syntax(self.parser, options)
 
1215
        commands = commands_from_options(options)
 
1216
        self.assertEqual(len(commands), 1)
 
1217
        command = commands[0]
 
1218
        self.assertIsInstance(command, command_cls)
 
1219
        for key, value in cmd_attrs.items():
 
1220
            self.assertEqual(getattr(command, key), value)
 
1221
    def test_print_table(self):
 
1222
        self.assert_command_from_args([], PrintTableCmd,
 
1223
                                      verbose=False)
 
1224
 
 
1225
    def test_print_table_verbose(self):
 
1226
        self.assert_command_from_args(["--verbose"], PrintTableCmd,
 
1227
                                      verbose=True)
 
1228
 
 
1229
    def test_print_table_verbose_short(self):
 
1230
        self.assert_command_from_args(["-v"], PrintTableCmd,
 
1231
                                      verbose=True)
 
1232
 
 
1233
    def test_enable(self):
 
1234
        self.assert_command_from_args(["--enable", "foo"], EnableCmd)
 
1235
 
 
1236
    def test_enable_short(self):
 
1237
        self.assert_command_from_args(["-e", "foo"], EnableCmd)
 
1238
 
 
1239
    def test_disable(self):
 
1240
        self.assert_command_from_args(["--disable", "foo"],
 
1241
                                      DisableCmd)
 
1242
 
 
1243
    def test_disable_short(self):
 
1244
        self.assert_command_from_args(["-d", "foo"], DisableCmd)
 
1245
 
 
1246
    def test_bump_timeout(self):
 
1247
        self.assert_command_from_args(["--bump-timeout", "foo"],
 
1248
                                      BumpTimeoutCmd)
 
1249
 
 
1250
    def test_bump_timeout_short(self):
 
1251
        self.assert_command_from_args(["-b", "foo"], BumpTimeoutCmd)
 
1252
 
 
1253
    def test_start_checker(self):
 
1254
        self.assert_command_from_args(["--start-checker", "foo"],
 
1255
                                      StartCheckerCmd)
 
1256
 
 
1257
    def test_stop_checker(self):
 
1258
        self.assert_command_from_args(["--stop-checker", "foo"],
 
1259
                                      StopCheckerCmd)
 
1260
 
 
1261
    def test_remove(self):
 
1262
        self.assert_command_from_args(["--remove", "foo"],
 
1263
                                      RemoveCmd)
 
1264
 
 
1265
    def test_remove_short(self):
 
1266
        self.assert_command_from_args(["-r", "foo"], RemoveCmd)
 
1267
 
 
1268
    def test_checker(self):
 
1269
        self.assert_command_from_args(["--checker", ":", "foo"],
 
1270
                                      SetCheckerCmd, value_to_set=":")
 
1271
 
 
1272
    def test_checker_empty(self):
 
1273
        self.assert_command_from_args(["--checker", "", "foo"],
 
1274
                                      SetCheckerCmd, value_to_set="")
 
1275
 
 
1276
    def test_checker_short(self):
 
1277
        self.assert_command_from_args(["-c", ":", "foo"],
 
1278
                                      SetCheckerCmd, value_to_set=":")
 
1279
 
 
1280
    def test_timeout(self):
 
1281
        self.assert_command_from_args(["--timeout", "PT5M", "foo"],
 
1282
                                      SetTimeoutCmd,
 
1283
                                      value_to_set=300000)
 
1284
 
 
1285
    def test_timeout_short(self):
 
1286
        self.assert_command_from_args(["-t", "PT5M", "foo"],
 
1287
                                      SetTimeoutCmd,
 
1288
                                      value_to_set=300000)
 
1289
 
 
1290
    def test_extended_timeout(self):
 
1291
        self.assert_command_from_args(["--extended-timeout", "PT15M",
 
1292
                                       "foo"],
 
1293
                                      SetExtendedTimeoutCmd,
 
1294
                                      value_to_set=900000)
 
1295
 
 
1296
    def test_interval(self):
 
1297
        self.assert_command_from_args(["--interval", "PT2M", "foo"],
 
1298
                                      SetIntervalCmd,
 
1299
                                      value_to_set=120000)
 
1300
 
 
1301
    def test_interval_short(self):
 
1302
        self.assert_command_from_args(["-i", "PT2M", "foo"],
 
1303
                                      SetIntervalCmd,
 
1304
                                      value_to_set=120000)
 
1305
 
 
1306
    def test_approve_by_default(self):
 
1307
        self.assert_command_from_args(["--approve-by-default", "foo"],
 
1308
                                      ApproveByDefaultCmd)
 
1309
 
 
1310
    def test_deny_by_default(self):
 
1311
        self.assert_command_from_args(["--deny-by-default", "foo"],
 
1312
                                      DenyByDefaultCmd)
 
1313
 
 
1314
    def test_approval_delay(self):
 
1315
        self.assert_command_from_args(["--approval-delay", "PT30S",
 
1316
                                       "foo"], SetApprovalDelayCmd,
 
1317
                                      value_to_set=30000)
 
1318
 
 
1319
    def test_approval_duration(self):
 
1320
        self.assert_command_from_args(["--approval-duration", "PT1S",
 
1321
                                       "foo"], SetApprovalDurationCmd,
 
1322
                                      value_to_set=1000)
 
1323
 
 
1324
    def test_host(self):
 
1325
        self.assert_command_from_args(["--host", "foo.example.org",
 
1326
                                       "foo"], SetHostCmd,
 
1327
                                      value_to_set="foo.example.org")
 
1328
 
 
1329
    def test_host_short(self):
 
1330
        self.assert_command_from_args(["-H", "foo.example.org",
 
1331
                                       "foo"], SetHostCmd,
 
1332
                                      value_to_set="foo.example.org")
 
1333
 
 
1334
    def test_secret_devnull(self):
 
1335
        self.assert_command_from_args(["--secret", os.path.devnull,
 
1336
                                       "foo"], SetSecretCmd,
 
1337
                                      value_to_set=b"")
 
1338
 
 
1339
    def test_secret_tempfile(self):
 
1340
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
 
1341
            value = b"secret\0xyzzy\nbar"
 
1342
            f.write(value)
 
1343
            f.seek(0)
 
1344
            self.assert_command_from_args(["--secret", f.name,
 
1345
                                           "foo"], SetSecretCmd,
 
1346
                                          value_to_set=value)
 
1347
 
 
1348
    def test_secret_devnull_short(self):
 
1349
        self.assert_command_from_args(["-s", os.path.devnull, "foo"],
 
1350
                                      SetSecretCmd, value_to_set=b"")
 
1351
 
 
1352
    def test_secret_tempfile_short(self):
 
1353
        with tempfile.NamedTemporaryFile(mode="r+b") as f:
 
1354
            value = b"secret\0xyzzy\nbar"
 
1355
            f.write(value)
 
1356
            f.seek(0)
 
1357
            self.assert_command_from_args(["-s", f.name, "foo"],
 
1358
                                          SetSecretCmd,
 
1359
                                          value_to_set=value)
 
1360
 
 
1361
    def test_approve(self):
 
1362
        self.assert_command_from_args(["--approve", "foo"],
 
1363
                                      ApproveCmd)
 
1364
 
 
1365
    def test_approve_short(self):
 
1366
        self.assert_command_from_args(["-A", "foo"], ApproveCmd)
 
1367
 
 
1368
    def test_deny(self):
 
1369
        self.assert_command_from_args(["--deny", "foo"], DenyCmd)
 
1370
 
 
1371
    def test_deny_short(self):
 
1372
        self.assert_command_from_args(["-D", "foo"], DenyCmd)
 
1373
 
 
1374
    def test_dump_json(self):
 
1375
        self.assert_command_from_args(["--dump-json"], DumpJSONCmd)
 
1376
 
 
1377
    def test_is_enabled(self):
 
1378
        self.assert_command_from_args(["--is-enabled", "foo"],
 
1379
                                      IsEnabledCmd)
 
1380
 
 
1381
    def test_is_enabled_short(self):
 
1382
        self.assert_command_from_args(["-V", "foo"], IsEnabledCmd)
 
1383
 
 
1384
    def test_deny_before_remove(self):
 
1385
        options = self.parser.parse_args(["--deny", "--remove", "foo"])
 
1386
        check_option_syntax(self.parser, options)
 
1387
        commands = commands_from_options(options)
 
1388
        self.assertEqual(len(commands), 2)
 
1389
        self.assertIsInstance(commands[0], DenyCmd)
 
1390
        self.assertIsInstance(commands[1], RemoveCmd)
 
1391
 
 
1392
    def test_deny_before_remove_reversed(self):
 
1393
        options = self.parser.parse_args(["--remove", "--deny", "--all"])
 
1394
        check_option_syntax(self.parser, options)
 
1395
        commands = commands_from_options(options)
 
1396
        self.assertEqual(len(commands), 2)
 
1397
        self.assertIsInstance(commands[0], DenyCmd)
 
1398
        self.assertIsInstance(commands[1], RemoveCmd)
 
1399
 
 
1400
 
 
1401
class Test_check_option_syntax(unittest.TestCase):
 
1402
    # This mostly corresponds to the definition from has_actions() in
 
1403
    # check_option_syntax()
 
1404
    actions = {
 
1405
        # The actual values set here are not that important, but we do
 
1406
        # at least stick to the correct types, even though they are
 
1407
        # never used
 
1408
        "enable": True,
 
1409
        "disable": True,
 
1410
        "bump_timeout": True,
 
1411
        "start_checker": True,
 
1412
        "stop_checker": True,
 
1413
        "is_enabled": True,
 
1414
        "remove": True,
 
1415
        "checker": "x",
 
1416
        "timeout": datetime.timedelta(),
 
1417
        "extended_timeout": datetime.timedelta(),
 
1418
        "interval": datetime.timedelta(),
 
1419
        "approved_by_default": True,
 
1420
        "approval_delay": datetime.timedelta(),
 
1421
        "approval_duration": datetime.timedelta(),
 
1422
        "host": "x",
 
1423
        "secret": io.BytesIO(b"x"),
 
1424
        "approve": True,
 
1425
        "deny": True,
 
1426
    }
 
1427
 
 
1428
    def setUp(self):
 
1429
        self.parser = argparse.ArgumentParser()
 
1430
        add_command_line_options(self.parser)
 
1431
 
 
1432
    @contextlib.contextmanager
 
1433
    def assertParseError(self):
 
1434
        with self.assertRaises(SystemExit) as e:
 
1435
            with self.temporarily_suppress_stderr():
 
1436
                yield
 
1437
        # Exit code from argparse is guaranteed to be "2".  Reference:
 
1438
        # https://docs.python.org/3/library/argparse.html#exiting-methods
 
1439
        self.assertEqual(e.exception.code, 2)
 
1440
 
 
1441
    @staticmethod
 
1442
    @contextlib.contextmanager
 
1443
    def temporarily_suppress_stderr():
 
1444
        null = os.open(os.path.devnull, os.O_RDWR)
 
1445
        stderrcopy = os.dup(sys.stderr.fileno())
 
1446
        os.dup2(null, sys.stderr.fileno())
 
1447
        os.close(null)
 
1448
        try:
 
1449
            yield
 
1450
        finally:
 
1451
            # restore stderr
 
1452
            os.dup2(stderrcopy, sys.stderr.fileno())
 
1453
            os.close(stderrcopy)
 
1454
 
 
1455
    def check_option_syntax(self, options):
 
1456
        check_option_syntax(self.parser, options)
 
1457
 
 
1458
    def test_actions_requires_client_or_all(self):
 
1459
        for action, value in self.actions.items():
 
1460
            options = self.parser.parse_args()
 
1461
            setattr(options, action, value)
 
1462
            with self.assertParseError():
 
1463
                self.check_option_syntax(options)
 
1464
 
 
1465
    def test_actions_conflicts_with_verbose(self):
 
1466
        for action, value in self.actions.items():
 
1467
            options = self.parser.parse_args()
 
1468
            setattr(options, action, value)
 
1469
            options.verbose = True
 
1470
            with self.assertParseError():
 
1471
                self.check_option_syntax(options)
 
1472
 
 
1473
    def test_dump_json_conflicts_with_verbose(self):
 
1474
        options = self.parser.parse_args()
 
1475
        options.dump_json = True
 
1476
        options.verbose = True
 
1477
        with self.assertParseError():
 
1478
            self.check_option_syntax(options)
 
1479
 
 
1480
    def test_dump_json_conflicts_with_action(self):
 
1481
        for action, value in self.actions.items():
 
1482
            options = self.parser.parse_args()
 
1483
            setattr(options, action, value)
 
1484
            options.dump_json = True
 
1485
            with self.assertParseError():
 
1486
                self.check_option_syntax(options)
 
1487
 
 
1488
    def test_all_can_not_be_alone(self):
 
1489
        options = self.parser.parse_args()
 
1490
        options.all = True
 
1491
        with self.assertParseError():
 
1492
            self.check_option_syntax(options)
 
1493
 
 
1494
    def test_all_is_ok_with_any_action(self):
 
1495
        for action, value in self.actions.items():
 
1496
            options = self.parser.parse_args()
 
1497
            setattr(options, action, value)
 
1498
            options.all = True
 
1499
            self.check_option_syntax(options)
 
1500
 
 
1501
    def test_is_enabled_fails_without_client(self):
 
1502
        options = self.parser.parse_args()
 
1503
        options.is_enabled = True
 
1504
        with self.assertParseError():
 
1505
            self.check_option_syntax(options)
 
1506
 
 
1507
    def test_is_enabled_works_with_one_client(self):
 
1508
        options = self.parser.parse_args()
 
1509
        options.is_enabled = True
 
1510
        options.client = ["foo"]
 
1511
        self.check_option_syntax(options)
 
1512
 
 
1513
    def test_is_enabled_fails_with_two_clients(self):
 
1514
        options = self.parser.parse_args()
 
1515
        options.is_enabled = True
 
1516
        options.client = ["foo", "barbar"]
 
1517
        with self.assertParseError():
 
1518
            self.check_option_syntax(options)
 
1519
 
 
1520
    def test_remove_can_only_be_combined_with_action_deny(self):
 
1521
        for action, value in self.actions.items():
 
1522
            if action in {"remove", "deny"}:
 
1523
                continue
 
1524
            options = self.parser.parse_args()
 
1525
            setattr(options, action, value)
 
1526
            options.all = True
 
1527
            options.remove = True
 
1528
            with self.assertParseError():
 
1529
                self.check_option_syntax(options)
834
1530
 
835
1531
 
836
1532