/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-17 10:59:24 UTC
  • Revision ID: teddy@recompile.se-20190317105924-3o9xb4pl8tld3rnp
mandos-ctl: Refactor

* mandos-ctl: Move all command classes into a class acting as a
              namespace.  Remove "Cmd" suffix from command class
              names.  All users of commands changed.  Also unify all
              tests of base commands into one test class.
  (command): New.
  (TestBaseCommands): New.
  (TestIsEnableCmd, TestApproveCmd, TestDenyCmd, TestRemoveCmd,
  TestDumpJSONCmd, TestPrintTableCmd): Remove; move tests into
                                       TestBaseCommands.

Show diffs side-by-side

added added

removed removed

Lines of Context:
1
 
#!/usr/bin/python3 -bbI
2
 
# -*- after-save-hook: (lambda () (let ((command (if (fboundp 'file-local-name) (file-local-name (buffer-file-name)) (or (file-remote-p (buffer-file-name) 'localname) (buffer-file-name))))) (if (= (progn (if (get-buffer "*Test*") (kill-buffer "*Test*")) (process-file-shell-command (format "%s --check" (shell-quote-argument command)) nil "*Test*")) 0) (let ((w (get-buffer-window "*Test*"))) (if w (delete-window w))) (progn (with-current-buffer "*Test*" (compilation-mode)) (display-buffer "*Test*" '(display-buffer-in-side-window)))))); coding: utf-8 -*-
 
1
#!/usr/bin/python
 
2
# -*- mode: python; coding: utf-8; after-save-hook: (lambda () (let ((command (if (and (boundp 'tramp-file-name-structure) (string-match (car tramp-file-name-structure) (buffer-file-name))) (tramp-file-name-localname (tramp-dissect-file-name (buffer-file-name))) (buffer-file-name)))) (if (= (shell-command (format "%s --check" (shell-quote-argument command)) "*Test*") 0) (let ((w (get-buffer-window "*Test*"))) (if w (delete-window w)) (kill-buffer "*Test*")) (display-buffer "*Test*")))); -*-
3
3
#
4
4
# Mandos Monitor - Control and monitor the Mandos server
5
5
#
46
46
import tempfile
47
47
import contextlib
48
48
 
49
 
if sys.version_info.major == 2:
50
 
    __metaclass__ = type
51
 
    str = unicode
52
 
 
53
 
class gi:
54
 
    """Dummy gi module, for the tests"""
55
 
    class repository:
56
 
        class GLib:
57
 
            class Error(Exception):
58
 
                pass
59
 
dbussy = None
60
 
ravel = None
61
 
dbus_python = None
62
 
pydbus = None
63
 
 
64
 
try:
65
 
    import dbussy
66
 
    import ravel
67
 
except ImportError:
68
 
    try:
69
 
        import pydbus
70
 
        import gi
71
 
    except ImportError:
72
 
        import dbus as dbus_python
73
 
 
 
49
import dbus
74
50
 
75
51
# Show warnings by default
76
52
if not sys.warnoptions:
84
60
logging.captureWarnings(True)   # Show warnings via the logging system
85
61
 
86
62
if sys.version_info.major == 2:
87
 
    import StringIO
88
 
    io.StringIO = StringIO.StringIO
 
63
    str = unicode
89
64
 
90
65
locale.setlocale(locale.LC_ALL, "")
91
66
 
92
 
version = "1.8.9"
 
67
dbus_busname_domain = "se.recompile"
 
68
dbus_busname = dbus_busname_domain + ".Mandos"
 
69
server_dbus_path = "/"
 
70
server_dbus_interface = dbus_busname_domain + ".Mandos"
 
71
client_dbus_interface = dbus_busname_domain + ".Mandos.Client"
 
72
del dbus_busname_domain
 
73
version = "1.8.3"
 
74
 
 
75
 
 
76
try:
 
77
    dbus.OBJECT_MANAGER_IFACE
 
78
except AttributeError:
 
79
    dbus.OBJECT_MANAGER_IFACE = "org.freedesktop.DBus.ObjectManager"
93
80
 
94
81
 
95
82
def main():
96
83
    parser = argparse.ArgumentParser()
 
84
 
97
85
    add_command_line_options(parser)
98
86
 
99
87
    options = parser.parse_args()
 
88
 
100
89
    check_option_syntax(parser, options)
101
90
 
102
91
    clientnames = options.client
104
93
    if options.debug:
105
94
        log.setLevel(logging.DEBUG)
106
95
 
107
 
    if dbussy is not None and ravel is not None:
108
 
        bus = dbussy_adapter.CachingBus(dbussy, ravel)
109
 
    elif pydbus is not None:
110
 
        bus = pydbus_adapter.CachingBus(pydbus)
111
 
    else:
112
 
        bus = dbus_python_adapter.CachingBus(dbus_python)
113
 
 
114
 
    try:
115
 
        all_clients = bus.get_clients_and_properties()
116
 
    except dbus.ConnectFailed as e:
117
 
        log.critical("Could not connect to Mandos server: %s", e)
118
 
        sys.exit(1)
119
 
    except dbus.Error as e:
120
 
        log.critical(
121
 
            "Failed to access Mandos server through D-Bus:\n%s", e)
122
 
        sys.exit(1)
 
96
    bus = dbus.SystemBus()
 
97
 
 
98
    mandos_dbus_object = get_mandos_dbus_object(bus)
 
99
 
 
100
    mandos_serv = dbus.Interface(
 
101
        mandos_dbus_object, dbus_interface=server_dbus_interface)
 
102
    mandos_serv_object_manager = dbus.Interface(
 
103
        mandos_dbus_object, dbus_interface=dbus.OBJECT_MANAGER_IFACE)
 
104
 
 
105
    managed_objects = get_managed_objects(mandos_serv_object_manager)
 
106
 
 
107
    all_clients = {}
 
108
    for path, ifs_and_props in managed_objects.items():
 
109
        try:
 
110
            all_clients[path] = ifs_and_props[client_dbus_interface]
 
111
        except KeyError:
 
112
            pass
123
113
 
124
114
    # Compile dict of (clientpath: properties) to process
125
115
    if not clientnames:
138
128
    commands = commands_from_options(options)
139
129
 
140
130
    for command in commands:
141
 
        command.run(clients, bus)
 
131
        command.run(clients, bus, mandos_serv)
142
132
 
143
133
 
144
134
def add_command_line_options(parser):
149
139
                        help="Select all clients")
150
140
    parser.add_argument("-v", "--verbose", action="store_true",
151
141
                        help="Print all fields")
152
 
    parser.add_argument("-j", "--dump-json", dest="commands",
153
 
                        action="append_const", default=[],
154
 
                        const=command.DumpJSON(),
 
142
    parser.add_argument("-j", "--dump-json", action="store_true",
155
143
                        help="Dump client data in JSON format")
156
144
    enable_disable = parser.add_mutually_exclusive_group()
157
 
    enable_disable.add_argument("-e", "--enable", dest="commands",
158
 
                                action="append_const", default=[],
159
 
                                const=command.Enable(),
 
145
    enable_disable.add_argument("-e", "--enable", action="store_true",
160
146
                                help="Enable client")
161
 
    enable_disable.add_argument("-d", "--disable", dest="commands",
162
 
                                action="append_const", default=[],
163
 
                                const=command.Disable(),
 
147
    enable_disable.add_argument("-d", "--disable",
 
148
                                action="store_true",
164
149
                                help="disable client")
165
 
    parser.add_argument("-b", "--bump-timeout", dest="commands",
166
 
                        action="append_const", default=[],
167
 
                        const=command.BumpTimeout(),
 
150
    parser.add_argument("-b", "--bump-timeout", action="store_true",
168
151
                        help="Bump timeout for client")
169
152
    start_stop_checker = parser.add_mutually_exclusive_group()
170
153
    start_stop_checker.add_argument("--start-checker",
171
 
                                    dest="commands",
172
 
                                    action="append_const", default=[],
173
 
                                    const=command.StartChecker(),
 
154
                                    action="store_true",
174
155
                                    help="Start checker for client")
175
 
    start_stop_checker.add_argument("--stop-checker", dest="commands",
176
 
                                    action="append_const", default=[],
177
 
                                    const=command.StopChecker(),
 
156
    start_stop_checker.add_argument("--stop-checker",
 
157
                                    action="store_true",
178
158
                                    help="Stop checker for client")
179
 
    parser.add_argument("-V", "--is-enabled", dest="commands",
180
 
                        action="append_const", default=[],
181
 
                        const=command.IsEnabled(),
 
159
    parser.add_argument("-V", "--is-enabled", action="store_true",
182
160
                        help="Check if client is enabled")
183
 
    parser.add_argument("-r", "--remove", dest="commands",
184
 
                        action="append_const", default=[],
185
 
                        const=command.Remove(),
 
161
    parser.add_argument("-r", "--remove", action="store_true",
186
162
                        help="Remove client")
187
 
    parser.add_argument("-c", "--checker", dest="commands",
188
 
                        action="append", default=[],
189
 
                        metavar="COMMAND", type=command.SetChecker,
 
163
    parser.add_argument("-c", "--checker",
190
164
                        help="Set checker command for client")
191
 
    parser.add_argument(
192
 
        "-t", "--timeout", dest="commands", action="append",
193
 
        default=[], metavar="TIME",
194
 
        type=command.SetTimeout.argparse(string_to_delta),
195
 
        help="Set timeout for client")
196
 
    parser.add_argument(
197
 
        "--extended-timeout", dest="commands", action="append",
198
 
        default=[], metavar="TIME",
199
 
        type=command.SetExtendedTimeout.argparse(string_to_delta),
200
 
        help="Set extended timeout for client")
201
 
    parser.add_argument(
202
 
        "-i", "--interval", dest="commands", action="append",
203
 
        default=[], metavar="TIME",
204
 
        type=command.SetInterval.argparse(string_to_delta),
205
 
        help="Set checker interval for client")
 
165
    parser.add_argument("-t", "--timeout", type=string_to_delta,
 
166
                        help="Set timeout for client")
 
167
    parser.add_argument("--extended-timeout", type=string_to_delta,
 
168
                        help="Set extended timeout for client")
 
169
    parser.add_argument("-i", "--interval", type=string_to_delta,
 
170
                        help="Set checker interval for client")
206
171
    approve_deny_default = parser.add_mutually_exclusive_group()
207
172
    approve_deny_default.add_argument(
208
 
        "--approve-by-default", dest="commands",
209
 
        action="append_const", default=[],
210
 
        const=command.ApproveByDefault(),
 
173
        "--approve-by-default", action="store_true",
 
174
        default=None, dest="approved_by_default",
211
175
        help="Set client to be approved by default")
212
176
    approve_deny_default.add_argument(
213
 
        "--deny-by-default", dest="commands",
214
 
        action="append_const", default=[],
215
 
        const=command.DenyByDefault(),
 
177
        "--deny-by-default", action="store_false",
 
178
        dest="approved_by_default",
216
179
        help="Set client to be denied by default")
217
 
    parser.add_argument(
218
 
        "--approval-delay", dest="commands", action="append",
219
 
        default=[], metavar="TIME",
220
 
        type=command.SetApprovalDelay.argparse(string_to_delta),
221
 
        help="Set delay before client approve/deny")
222
 
    parser.add_argument(
223
 
        "--approval-duration", dest="commands", action="append",
224
 
        default=[], metavar="TIME",
225
 
        type=command.SetApprovalDuration.argparse(string_to_delta),
226
 
        help="Set duration of one client approval")
227
 
    parser.add_argument("-H", "--host", dest="commands",
228
 
                        action="append", default=[], metavar="STRING",
229
 
                        type=command.SetHost,
230
 
                        help="Set host for client")
231
 
    parser.add_argument(
232
 
        "-s", "--secret", dest="commands", action="append",
233
 
        default=[], metavar="FILENAME",
234
 
        type=command.SetSecret.argparse(argparse.FileType(mode="rb")),
235
 
        help="Set password blob (file) for client")
 
180
    parser.add_argument("--approval-delay", type=string_to_delta,
 
181
                        help="Set delay before client approve/deny")
 
182
    parser.add_argument("--approval-duration", type=string_to_delta,
 
183
                        help="Set duration of one client approval")
 
184
    parser.add_argument("-H", "--host", help="Set host for client")
 
185
    parser.add_argument("-s", "--secret",
 
186
                        type=argparse.FileType(mode="rb"),
 
187
                        help="Set password blob (file) for client")
236
188
    approve_deny = parser.add_mutually_exclusive_group()
237
189
    approve_deny.add_argument(
238
 
        "-A", "--approve", dest="commands", action="append_const",
239
 
        default=[], const=command.Approve(),
 
190
        "-A", "--approve", action="store_true",
240
191
        help="Approve any current client request")
241
 
    approve_deny.add_argument("-D", "--deny", dest="commands",
242
 
                              action="append_const", default=[],
243
 
                              const=command.Deny(),
 
192
    approve_deny.add_argument("-D", "--deny", action="store_true",
244
193
                              help="Deny any current client request")
245
194
    parser.add_argument("--debug", action="store_true",
246
195
                        help="Debug mode (show D-Bus commands)")
263
212
def rfc3339_duration_to_delta(duration):
264
213
    """Parse an RFC 3339 "duration" and return a datetime.timedelta
265
214
 
266
 
    >>> rfc3339_duration_to_delta("P7D") == datetime.timedelta(7)
267
 
    True
268
 
    >>> rfc3339_duration_to_delta("PT60S") == datetime.timedelta(0, 60)
269
 
    True
270
 
    >>> rfc3339_duration_to_delta("PT60M") == datetime.timedelta(hours=1)
271
 
    True
272
 
    >>> # 60 months
273
 
    >>> rfc3339_duration_to_delta("P60M") == datetime.timedelta(1680)
274
 
    True
275
 
    >>> rfc3339_duration_to_delta("PT24H") == datetime.timedelta(1)
276
 
    True
277
 
    >>> rfc3339_duration_to_delta("P1W") == datetime.timedelta(7)
278
 
    True
279
 
    >>> rfc3339_duration_to_delta("PT5M30S") == datetime.timedelta(0, 330)
280
 
    True
281
 
    >>> rfc3339_duration_to_delta("P1DT3M20S") == datetime.timedelta(1, 200)
282
 
    True
 
215
    >>> rfc3339_duration_to_delta("P7D")
 
216
    datetime.timedelta(7)
 
217
    >>> rfc3339_duration_to_delta("PT60S")
 
218
    datetime.timedelta(0, 60)
 
219
    >>> rfc3339_duration_to_delta("PT60M")
 
220
    datetime.timedelta(0, 3600)
 
221
    >>> rfc3339_duration_to_delta("P60M")
 
222
    datetime.timedelta(1680)
 
223
    >>> rfc3339_duration_to_delta("PT24H")
 
224
    datetime.timedelta(1)
 
225
    >>> rfc3339_duration_to_delta("P1W")
 
226
    datetime.timedelta(7)
 
227
    >>> rfc3339_duration_to_delta("PT5M30S")
 
228
    datetime.timedelta(0, 330)
 
229
    >>> rfc3339_duration_to_delta("P1DT3M20S")
 
230
    datetime.timedelta(1, 200)
283
231
    >>> # Can not be empty:
284
232
    >>> rfc3339_duration_to_delta("")
285
233
    Traceback (most recent call last):
395
343
    """Parse an interval string as documented by Mandos before 1.6.1,
396
344
    and return a datetime.timedelta
397
345
 
398
 
    >>> parse_pre_1_6_1_interval('7d') == datetime.timedelta(days=7)
399
 
    True
400
 
    >>> parse_pre_1_6_1_interval('60s') == datetime.timedelta(0, 60)
401
 
    True
402
 
    >>> parse_pre_1_6_1_interval('60m') == datetime.timedelta(hours=1)
403
 
    True
404
 
    >>> parse_pre_1_6_1_interval('24h') == datetime.timedelta(days=1)
405
 
    True
406
 
    >>> parse_pre_1_6_1_interval('1w') == datetime.timedelta(days=7)
407
 
    True
408
 
    >>> parse_pre_1_6_1_interval('5m 30s') == datetime.timedelta(0, 330)
409
 
    True
410
 
    >>> parse_pre_1_6_1_interval('') == datetime.timedelta(0)
411
 
    True
 
346
    >>> parse_pre_1_6_1_interval('7d')
 
347
    datetime.timedelta(7)
 
348
    >>> parse_pre_1_6_1_interval('60s')
 
349
    datetime.timedelta(0, 60)
 
350
    >>> parse_pre_1_6_1_interval('60m')
 
351
    datetime.timedelta(0, 3600)
 
352
    >>> parse_pre_1_6_1_interval('24h')
 
353
    datetime.timedelta(1)
 
354
    >>> parse_pre_1_6_1_interval('1w')
 
355
    datetime.timedelta(7)
 
356
    >>> parse_pre_1_6_1_interval('5m 30s')
 
357
    datetime.timedelta(0, 330)
 
358
    >>> parse_pre_1_6_1_interval('')
 
359
    datetime.timedelta(0)
412
360
    >>> # Ignore unknown characters, allow any order and repetitions
413
 
    >>> parse_pre_1_6_1_interval('2dxy7zz11y3m5m') == datetime.timedelta(2, 480, 18000)
414
 
    True
 
361
    >>> parse_pre_1_6_1_interval('2dxy7zz11y3m5m')
 
362
    datetime.timedelta(2, 480, 18000)
415
363
 
416
364
    """
417
365
 
438
386
    """Apply additional restrictions on options, not expressible in
439
387
argparse"""
440
388
 
441
 
    def has_commands(options, commands=None):
442
 
        if commands is None:
443
 
            commands = (command.Enable,
444
 
                        command.Disable,
445
 
                        command.BumpTimeout,
446
 
                        command.StartChecker,
447
 
                        command.StopChecker,
448
 
                        command.IsEnabled,
449
 
                        command.Remove,
450
 
                        command.SetChecker,
451
 
                        command.SetTimeout,
452
 
                        command.SetExtendedTimeout,
453
 
                        command.SetInterval,
454
 
                        command.ApproveByDefault,
455
 
                        command.DenyByDefault,
456
 
                        command.SetApprovalDelay,
457
 
                        command.SetApprovalDuration,
458
 
                        command.SetHost,
459
 
                        command.SetSecret,
460
 
                        command.Approve,
461
 
                        command.Deny)
462
 
        return any(isinstance(cmd, commands)
463
 
                   for cmd in options.commands)
 
389
    def has_actions(options):
 
390
        return any((options.enable,
 
391
                    options.disable,
 
392
                    options.bump_timeout,
 
393
                    options.start_checker,
 
394
                    options.stop_checker,
 
395
                    options.is_enabled,
 
396
                    options.remove,
 
397
                    options.checker is not None,
 
398
                    options.timeout is not None,
 
399
                    options.extended_timeout is not None,
 
400
                    options.interval is not None,
 
401
                    options.approved_by_default is not None,
 
402
                    options.approval_delay is not None,
 
403
                    options.approval_duration is not None,
 
404
                    options.host is not None,
 
405
                    options.secret is not None,
 
406
                    options.approve,
 
407
                    options.deny))
464
408
 
465
 
    if has_commands(options) and not (options.client or options.all):
 
409
    if has_actions(options) and not (options.client or options.all):
466
410
        parser.error("Options require clients names or --all.")
467
 
    if options.verbose and has_commands(options):
 
411
    if options.verbose and has_actions(options):
468
412
        parser.error("--verbose can only be used alone.")
469
 
    if (has_commands(options, (command.DumpJSON,))
470
 
        and (options.verbose or len(options.commands) > 1)):
 
413
    if options.dump_json and (options.verbose
 
414
                              or has_actions(options)):
471
415
        parser.error("--dump-json can only be used alone.")
472
 
    if options.all and not has_commands(options):
 
416
    if options.all and not has_actions(options):
473
417
        parser.error("--all requires an action.")
474
 
    if (has_commands(options, (command.IsEnabled,))
475
 
        and len(options.client) > 1):
 
418
    if options.is_enabled and len(options.client) > 1:
476
419
        parser.error("--is-enabled requires exactly one client")
477
 
    if (len(options.commands) > 1
478
 
        and has_commands(options, (command.Remove,))
479
 
        and not has_commands(options, (command.Deny,))):
480
 
        parser.error("--remove can only be combined with --deny")
481
 
 
482
 
 
483
 
class dbus:
484
 
 
485
 
    class SystemBus:
486
 
 
487
 
        object_manager_iface = "org.freedesktop.DBus.ObjectManager"
488
 
        def get_managed_objects(self, busname, objectpath):
489
 
            return self.call_method("GetManagedObjects", busname,
490
 
                                    objectpath,
491
 
                                    self.object_manager_iface)
492
 
 
493
 
        properties_iface = "org.freedesktop.DBus.Properties"
494
 
        def set_property(self, busname, objectpath, interface, key,
495
 
                         value):
496
 
            self.call_method("Set", busname, objectpath,
497
 
                             self.properties_iface, interface, key,
498
 
                             value)
499
 
 
500
 
    class MandosBus(SystemBus):
501
 
        busname_domain = "se.recompile"
502
 
        busname = busname_domain + ".Mandos"
503
 
        server_path = "/"
504
 
        server_interface = busname_domain + ".Mandos"
505
 
        client_interface = busname_domain + ".Mandos.Client"
506
 
        del busname_domain
507
 
 
508
 
        def get_clients_and_properties(self):
509
 
            managed_objects = self.get_managed_objects(
510
 
                self.busname, self.server_path)
511
 
            return {objpath: properties[self.client_interface]
512
 
                    for objpath, properties in managed_objects.items()
513
 
                    if self.client_interface in properties}
514
 
 
515
 
        def set_client_property(self, objectpath, key, value):
516
 
            return self.set_property(self.busname, objectpath,
517
 
                                     self.client_interface, key,
518
 
                                     value)
519
 
 
520
 
        def call_client_method(self, objectpath, method, *args):
521
 
            return self.call_method(method, self.busname, objectpath,
522
 
                                    self.client_interface, *args)
523
 
 
524
 
        def call_server_method(self, method, *args):
525
 
            return self.call_method(method, self.busname,
526
 
                                    self.server_path,
527
 
                                    self.server_interface, *args)
528
 
 
529
 
    class Error(Exception):
530
 
        pass
531
 
 
532
 
    class ConnectFailed(Error):
533
 
        pass
534
 
 
535
 
 
536
 
class dbus_python_adapter:
537
 
 
538
 
    class SystemBus(dbus.MandosBus):
539
 
        """Use dbus-python"""
540
 
 
541
 
        def __init__(self, module=dbus_python):
542
 
            self.dbus_python = module
543
 
            self.bus = self.dbus_python.SystemBus()
544
 
 
545
 
        @contextlib.contextmanager
546
 
        def convert_exception(self, exception_class=dbus.Error):
547
 
            try:
548
 
                yield
549
 
            except self.dbus_python.exceptions.DBusException as e:
550
 
                # This does what "raise from" would do
551
 
                exc = exception_class(*e.args)
552
 
                exc.__cause__ = e
553
 
                raise exc
554
 
 
555
 
        def call_method(self, methodname, busname, objectpath,
556
 
                        interface, *args):
557
 
            proxy_object = self.get_object(busname, objectpath)
558
 
            log.debug("D-Bus: %s:%s:%s.%s(%s)", busname, objectpath,
559
 
                      interface, methodname,
560
 
                      ", ".join(repr(a) for a in args))
561
 
            method = getattr(proxy_object, methodname)
562
 
            with self.convert_exception():
563
 
                with dbus_python_adapter.SilenceLogger(
564
 
                        "dbus.proxies"):
565
 
                    value = method(*args, dbus_interface=interface)
566
 
            return self.type_filter(value)
567
 
 
568
 
        def get_object(self, busname, objectpath):
569
 
            log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
570
 
                      busname, objectpath)
571
 
            with self.convert_exception(dbus.ConnectFailed):
572
 
                return self.bus.get_object(busname, objectpath)
573
 
 
574
 
        def type_filter(self, value):
575
 
            """Convert the most bothersome types to Python types"""
576
 
            if isinstance(value, self.dbus_python.Boolean):
577
 
                return bool(value)
578
 
            if isinstance(value, self.dbus_python.ObjectPath):
579
 
                return str(value)
580
 
            # Also recurse into dictionaries
581
 
            if isinstance(value, self.dbus_python.Dictionary):
582
 
                return {self.type_filter(key):
583
 
                        self.type_filter(subval)
584
 
                        for key, subval in value.items()}
585
 
            return value
586
 
 
587
 
        def set_client_property(self, objectpath, key, value):
588
 
            if key == "Secret":
589
 
                if not isinstance(value, bytes):
590
 
                    value = value.encode("utf-8")
591
 
                value = self.dbus_python.ByteArray(value)
592
 
            return self.set_property(self.busname, objectpath,
593
 
                                     self.client_interface, key,
594
 
                                     value)
595
 
 
596
 
    class SilenceLogger:
597
 
        "Simple context manager to silence a particular logger"
598
 
        def __init__(self, loggername):
599
 
            self.logger = logging.getLogger(loggername)
600
 
 
601
 
        def __enter__(self):
602
 
            self.logger.addFilter(self.nullfilter)
603
 
 
604
 
        class NullFilter(logging.Filter):
605
 
            def filter(self, record):
606
 
                return False
607
 
 
608
 
        nullfilter = NullFilter()
609
 
 
610
 
        def __exit__(self, exc_type, exc_val, exc_tb):
611
 
            self.logger.removeFilter(self.nullfilter)
612
 
 
613
 
 
614
 
    class CachingBus(SystemBus):
615
 
        """A caching layer for dbus_python_adapter.SystemBus"""
616
 
        def __init__(self, *args, **kwargs):
617
 
            self.object_cache = {}
618
 
            super(dbus_python_adapter.CachingBus,
619
 
                  self).__init__(*args, **kwargs)
620
 
        def get_object(self, busname, objectpath):
621
 
            try:
622
 
                return self.object_cache[(busname, objectpath)]
623
 
            except KeyError:
624
 
                new_object = super(
625
 
                    dbus_python_adapter.CachingBus,
626
 
                    self).get_object(busname, objectpath)
627
 
                self.object_cache[(busname, objectpath)]  = new_object
628
 
                return new_object
629
 
 
630
 
 
631
 
class pydbus_adapter:
632
 
    class SystemBus(dbus.MandosBus):
633
 
        def __init__(self, module=pydbus):
634
 
            self.pydbus = module
635
 
            self.bus = self.pydbus.SystemBus()
636
 
 
637
 
        @contextlib.contextmanager
638
 
        def convert_exception(self, exception_class=dbus.Error):
639
 
            try:
640
 
                yield
641
 
            except gi.repository.GLib.Error as e:
642
 
                # This does what "raise from" would do
643
 
                exc = exception_class(*e.args)
644
 
                exc.__cause__ = e
645
 
                raise exc
646
 
 
647
 
        def call_method(self, methodname, busname, objectpath,
648
 
                        interface, *args):
649
 
            proxy_object = self.get(busname, objectpath)
650
 
            log.debug("D-Bus: %s:%s:%s.%s(%s)", busname, objectpath,
651
 
                      interface, methodname,
652
 
                      ", ".join(repr(a) for a in args))
653
 
            method = getattr(proxy_object[interface], methodname)
654
 
            with self.convert_exception():
655
 
                return method(*args)
656
 
 
657
 
        def get(self, busname, objectpath):
658
 
            log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
659
 
                      busname, objectpath)
660
 
            with self.convert_exception(dbus.ConnectFailed):
661
 
                if sys.version_info.major <= 2:
662
 
                    with warnings.catch_warnings():
663
 
                        warnings.filterwarnings(
664
 
                            "ignore", "", DeprecationWarning,
665
 
                            r"^xml\.etree\.ElementTree$")
666
 
                        return self.bus.get(busname, objectpath)
667
 
                else:
668
 
                    return self.bus.get(busname, objectpath)
669
 
 
670
 
        def set_property(self, busname, objectpath, interface, key,
671
 
                         value):
672
 
            proxy_object = self.get(busname, objectpath)
673
 
            log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", busname,
674
 
                      objectpath, self.properties_iface, interface,
675
 
                      key, value)
676
 
            setattr(proxy_object[interface], key, value)
677
 
 
678
 
    class CachingBus(SystemBus):
679
 
        """A caching layer for pydbus_adapter.SystemBus"""
680
 
        def __init__(self, *args, **kwargs):
681
 
            self.object_cache = {}
682
 
            super(pydbus_adapter.CachingBus,
683
 
                  self).__init__(*args, **kwargs)
684
 
        def get(self, busname, objectpath):
685
 
            try:
686
 
                return self.object_cache[(busname, objectpath)]
687
 
            except KeyError:
688
 
                new_object = (super(pydbus_adapter.CachingBus, self)
689
 
                              .get(busname, objectpath))
690
 
                self.object_cache[(busname, objectpath)]  = new_object
691
 
                return new_object
692
 
 
693
 
 
694
 
class dbussy_adapter:
695
 
    class SystemBus(dbus.SystemBus):
696
 
        """Use DBussy"""
697
 
 
698
 
        def __init__(self, dbussy, ravel):
699
 
            self.dbussy = dbussy
700
 
            self.ravel = ravel
701
 
            self.bus = ravel.system_bus()
702
 
 
703
 
        @contextlib.contextmanager
704
 
        def convert_exception(self, exception_class=dbus.Error):
705
 
            try:
706
 
                yield
707
 
            except self.dbussy.DBusError as e:
708
 
                # This does what "raise from" would do
709
 
                exc = exception_class(*e.args)
710
 
                exc.__cause__ = e
711
 
                raise exc
712
 
 
713
 
        def call_method(self, methodname, busname, objectpath,
714
 
                        interface, *args):
715
 
            proxy_object = self.get_object(busname, objectpath)
716
 
            log.debug("D-Bus: %s:%s:%s.%s(%s)", busname, objectpath,
717
 
                      interface, methodname,
718
 
                      ", ".join(repr(a) for a in args))
719
 
            iface = proxy_object.get_interface(interface)
720
 
            method = getattr(iface, methodname)
721
 
            with self.convert_exception(dbus.Error):
722
 
                value =  method(*args)
723
 
            # DBussy returns values either as an empty list or as a
724
 
            # tuple: (signature, value)
725
 
            if value:
726
 
                return self.type_filter(value[0])
727
 
 
728
 
        def get_object(self, busname, objectpath):
729
 
            log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
730
 
                      busname, objectpath)
731
 
            with self.convert_exception(dbus.ConnectFailed):
732
 
                return self.bus[busname][objectpath]
733
 
 
734
 
        def type_filter(self, value):
735
 
            """Convert the most bothersome types to Python types"""
736
 
            if isinstance(value, tuple):
737
 
                if (len(value) == 2
738
 
                    and isinstance(value[0],
739
 
                                   self.dbussy.DBUS.Signature)):
740
 
                    return self.type_filter(value[1])
741
 
            elif isinstance(value, self.dbussy.DBUS.ObjectPath):
742
 
                return str(value)
743
 
            # Also recurse into dictionaries
744
 
            elif isinstance(value, dict):
745
 
                return {self.type_filter(key):
746
 
                        self.type_filter(subval)
747
 
                        for key, subval in value.items()}
748
 
            return value
749
 
 
750
 
        def set_property(self, busname, objectpath, interface, key,
751
 
                         value):
752
 
            proxy_object = self.get_object(busname, objectpath)
753
 
            log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", busname,
754
 
                      objectpath, self.properties_iface, interface,
755
 
                      key, value)
756
 
            if key == "Secret":
757
 
                # DBussy wants a Byte Array to be a sequence of
758
 
                # values, not a byte string
759
 
                value = tuple(value)
760
 
            setattr(proxy_object.get_interface(interface), key, value)
761
 
 
762
 
    class MandosBus(SystemBus, dbus.MandosBus):
763
 
        pass
764
 
 
765
 
    class CachingBus(MandosBus):
766
 
        """A caching layer for dbussy_adapter.MandosBus"""
767
 
        def __init__(self, *args, **kwargs):
768
 
            self.object_cache = {}
769
 
            super(dbussy_adapter.CachingBus, self).__init__(*args,
770
 
                                                            **kwargs)
771
 
        def get_object(self, busname, objectpath):
772
 
            try:
773
 
                return self.object_cache[(busname, objectpath)]
774
 
            except KeyError:
775
 
                new_object = super(
776
 
                    dbussy_adapter.CachingBus,
777
 
                    self).get_object(busname, objectpath)
778
 
                self.object_cache[(busname, objectpath)]  = new_object
779
 
                return new_object
 
420
    if options.remove:
 
421
        options.remove = False
 
422
        if has_actions(options) and not options.deny:
 
423
            parser.error("--remove can only be combined with --deny")
 
424
        options.remove = True
 
425
 
 
426
 
 
427
def get_mandos_dbus_object(bus):
 
428
    log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
 
429
              dbus_busname, server_dbus_path)
 
430
    with if_dbus_exception_log_with_exception_and_exit(
 
431
            "Could not connect to Mandos server: %s"):
 
432
        mandos_dbus_object = bus.get_object(dbus_busname,
 
433
                                            server_dbus_path)
 
434
    return mandos_dbus_object
 
435
 
 
436
 
 
437
@contextlib.contextmanager
 
438
def if_dbus_exception_log_with_exception_and_exit(*args, **kwargs):
 
439
    try:
 
440
        yield
 
441
    except dbus.exceptions.DBusException as e:
 
442
        log.critical(*(args + (e,)), **kwargs)
 
443
        sys.exit(1)
 
444
 
 
445
 
 
446
def get_managed_objects(object_manager):
 
447
    log.debug("D-Bus: %s:%s:%s.GetManagedObjects()", dbus_busname,
 
448
              server_dbus_path, dbus.OBJECT_MANAGER_IFACE)
 
449
    with if_dbus_exception_log_with_exception_and_exit(
 
450
            "Failed to access Mandos server through D-Bus:\n%s"):
 
451
        with SilenceLogger("dbus.proxies"):
 
452
            managed_objects = object_manager.GetManagedObjects()
 
453
    return managed_objects
 
454
 
 
455
 
 
456
class SilenceLogger(object):
 
457
    "Simple context manager to silence a particular logger"
 
458
    def __init__(self, loggername):
 
459
        self.logger = logging.getLogger(loggername)
 
460
 
 
461
    def __enter__(self):
 
462
        self.logger.addFilter(self.nullfilter)
 
463
        return self
 
464
 
 
465
    class NullFilter(logging.Filter):
 
466
        def filter(self, record):
 
467
            return False
 
468
 
 
469
    nullfilter = NullFilter()
 
470
 
 
471
    def __exit__(self, exc_type, exc_val, exc_tb):
 
472
        self.logger.removeFilter(self.nullfilter)
780
473
 
781
474
 
782
475
def commands_from_options(options):
783
476
 
784
 
    commands = list(options.commands)
785
 
 
786
 
    def find_cmd(cmd, commands):
787
 
        i = 0
788
 
        for i, c in enumerate(commands):
789
 
            if isinstance(c, cmd):
790
 
                return i
791
 
        return i+1
792
 
 
793
 
    # If command.Remove is present, move any instances of command.Deny
794
 
    # to occur ahead of command.Remove.
795
 
    index_of_remove = find_cmd(command.Remove, commands)
796
 
    before_remove = commands[:index_of_remove]
797
 
    after_remove = commands[index_of_remove:]
798
 
    cleaned_after = []
799
 
    for cmd in after_remove:
800
 
        if isinstance(cmd, command.Deny):
801
 
            before_remove.append(cmd)
 
477
    commands = []
 
478
 
 
479
    if options.is_enabled:
 
480
        commands.append(command.IsEnabled())
 
481
 
 
482
    if options.approve:
 
483
        commands.append(command.Approve())
 
484
 
 
485
    if options.deny:
 
486
        commands.append(command.Deny())
 
487
 
 
488
    if options.remove:
 
489
        commands.append(command.Remove())
 
490
 
 
491
    if options.dump_json:
 
492
        commands.append(command.DumpJSON())
 
493
 
 
494
    if options.enable:
 
495
        commands.append(command.Enable())
 
496
 
 
497
    if options.disable:
 
498
        commands.append(command.Disable())
 
499
 
 
500
    if options.bump_timeout:
 
501
        commands.append(command.BumpTimeout())
 
502
 
 
503
    if options.start_checker:
 
504
        commands.append(command.StartChecker())
 
505
 
 
506
    if options.stop_checker:
 
507
        commands.append(command.StopChecker())
 
508
 
 
509
    if options.approved_by_default is not None:
 
510
        if options.approved_by_default:
 
511
            commands.append(command.ApproveByDefault())
802
512
        else:
803
 
            cleaned_after.append(cmd)
804
 
    if cleaned_after != after_remove:
805
 
        commands = before_remove + cleaned_after
 
513
            commands.append(command.DenyByDefault())
 
514
 
 
515
    if options.checker is not None:
 
516
        commands.append(command.SetChecker(options.checker))
 
517
 
 
518
    if options.host is not None:
 
519
        commands.append(command.SetHost(options.host))
 
520
 
 
521
    if options.secret is not None:
 
522
        commands.append(command.SetSecret(options.secret))
 
523
 
 
524
    if options.timeout is not None:
 
525
        commands.append(command.SetTimeout(options.timeout))
 
526
 
 
527
    if options.extended_timeout:
 
528
        commands.append(
 
529
            command.SetExtendedTimeout(options.extended_timeout))
 
530
 
 
531
    if options.interval is not None:
 
532
        commands.append(command.SetInterval(options.interval))
 
533
 
 
534
    if options.approval_delay is not None:
 
535
        commands.append(
 
536
            command.SetApprovalDelay(options.approval_delay))
 
537
 
 
538
    if options.approval_duration is not None:
 
539
        commands.append(
 
540
            command.SetApprovalDuration(options.approval_duration))
806
541
 
807
542
    # If no command option has been given, show table of clients,
808
543
    # optionally verbosely
812
547
    return commands
813
548
 
814
549
 
815
 
class command:
 
550
class command(object):
816
551
    """A namespace for command classes"""
817
552
 
818
 
    class Base:
 
553
    class Base(object):
819
554
        """Abstract base class for commands"""
820
 
        def run(self, clients, bus=None):
 
555
        def run(self, clients, bus=None, mandos=None):
821
556
            """Normal commands should implement run_on_one_client(),
822
557
but commands which want to operate on all clients at the same time can
823
558
override this run() method instead.
824
559
"""
825
 
            self.bus = bus
826
 
            for client, properties in clients.items():
 
560
            self.mandos = mandos
 
561
            for clientpath, properties in clients.items():
 
562
                log.debug("D-Bus: Connect to: (busname=%r, path=%r)",
 
563
                          dbus_busname, str(clientpath))
 
564
                client = bus.get_object(dbus_busname, clientpath)
827
565
                self.run_on_one_client(client, properties)
828
566
 
829
567
 
830
568
    class IsEnabled(Base):
831
 
        def run(self, clients, bus=None):
832
 
            properties = next(iter(clients.values()))
833
 
            if properties["Enabled"]:
 
569
        def run(self, clients, bus=None, mandos=None):
 
570
            client, properties = next(iter(clients.items()))
 
571
            if self.is_enabled(client, properties):
834
572
                sys.exit(0)
835
573
            sys.exit(1)
 
574
        def is_enabled(self, client, properties):
 
575
            return properties["Enabled"]
836
576
 
837
577
 
838
578
    class Approve(Base):
839
579
        def run_on_one_client(self, client, properties):
840
 
            self.bus.call_client_method(client, "Approve", True)
 
580
            log.debug("D-Bus: %s:%s:%s.Approve(True)", dbus_busname,
 
581
                      client.__dbus_object_path__,
 
582
                      client_dbus_interface)
 
583
            client.Approve(dbus.Boolean(True),
 
584
                           dbus_interface=client_dbus_interface)
841
585
 
842
586
 
843
587
    class Deny(Base):
844
588
        def run_on_one_client(self, client, properties):
845
 
            self.bus.call_client_method(client, "Approve", False)
 
589
            log.debug("D-Bus: %s:%s:%s.Approve(False)", dbus_busname,
 
590
                      client.__dbus_object_path__,
 
591
                      client_dbus_interface)
 
592
            client.Approve(dbus.Boolean(False),
 
593
                           dbus_interface=client_dbus_interface)
846
594
 
847
595
 
848
596
    class Remove(Base):
849
 
        def run(self, clients, bus):
850
 
            for clientpath in frozenset(clients.keys()):
851
 
                bus.call_server_method("RemoveClient", clientpath)
 
597
        def run_on_one_client(self, client, properties):
 
598
            log.debug("D-Bus: %s:%s:%s.RemoveClient(%r)",
 
599
                      dbus_busname, server_dbus_path,
 
600
                      server_dbus_interface,
 
601
                      str(client.__dbus_object_path__))
 
602
            self.mandos.RemoveClient(client.__dbus_object_path__)
852
603
 
853
604
 
854
605
    class Output(Base):
862
613
                        "Checker", "ExtendedTimeout", "Expires",
863
614
                        "LastCheckerStatus")
864
615
 
 
616
        def run(self, clients, bus=None, mandos=None):
 
617
            print(self.output(clients.values()))
 
618
 
 
619
        def output(self, clients):
 
620
            raise NotImplementedError()
 
621
 
865
622
 
866
623
    class DumpJSON(Output):
867
 
        def run(self, clients, bus=None):
868
 
            data = {properties["Name"]:
869
 
                    {key: properties[key]
 
624
        def output(self, clients):
 
625
            data = {client["Name"]:
 
626
                    {key: self.dbus_boolean_to_bool(client[key])
870
627
                     for key in self.all_keywords}
871
 
                    for properties in clients.values()}
872
 
            print(json.dumps(data, indent=4, separators=(',', ': ')))
 
628
                    for client in clients}
 
629
            return json.dumps(data, indent=4, separators=(',', ': '))
 
630
 
 
631
        @staticmethod
 
632
        def dbus_boolean_to_bool(value):
 
633
            if isinstance(value, dbus.Boolean):
 
634
                value = bool(value)
 
635
            return value
873
636
 
874
637
 
875
638
    class PrintTable(Output):
876
639
        def __init__(self, verbose=False):
877
640
            self.verbose = verbose
878
641
 
879
 
        def run(self, clients, bus=None):
 
642
        def output(self, clients):
880
643
            default_keywords = ("Name", "Enabled", "Timeout",
881
644
                                "LastCheckedOK")
882
645
            keywords = default_keywords
883
646
            if self.verbose:
884
647
                keywords = self.all_keywords
885
 
            print(self.TableOfClients(clients.values(), keywords))
 
648
            return str(self.TableOfClients(clients, keywords))
886
649
 
887
 
        class TableOfClients:
 
650
        class TableOfClients(object):
888
651
            tableheaders = {
889
652
                "Name": "Name",
890
653
                "Enabled": "Enabled",
943
706
 
944
707
            @classmethod
945
708
            def valuetostring(cls, value, keyword):
946
 
                if isinstance(value, bool):
 
709
                if isinstance(value, dbus.Boolean):
947
710
                    return "Yes" if value else "No"
948
711
                if keyword in ("Timeout", "Interval", "ApprovalDelay",
949
712
                               "ApprovalDuration", "ExtendedTimeout"):
969
732
                                seconds=td.seconds % 60))
970
733
 
971
734
 
972
 
    class PropertySetter(Base):
 
735
    class Property(Base):
973
736
        "Abstract class for Actions for setting one client property"
974
737
 
975
 
        def run_on_one_client(self, client, properties=None):
 
738
        def run_on_one_client(self, client, properties):
976
739
            """Set the Client's D-Bus property"""
977
 
            self.bus.set_client_property(client, self.propname,
978
 
                                         self.value_to_set)
 
740
            log.debug("D-Bus: %s:%s:%s.Set(%r, %r, %r)", dbus_busname,
 
741
                      client.__dbus_object_path__,
 
742
                      dbus.PROPERTIES_IFACE, client_dbus_interface,
 
743
                      self.propname, self.value_to_set
 
744
                      if not isinstance(self.value_to_set,
 
745
                                        dbus.Boolean)
 
746
                      else bool(self.value_to_set))
 
747
            client.Set(client_dbus_interface, self.propname,
 
748
                       self.value_to_set,
 
749
                       dbus_interface=dbus.PROPERTIES_IFACE)
979
750
 
980
751
        @property
981
752
        def propname(self):
982
753
            raise NotImplementedError()
983
754
 
984
755
 
985
 
    class Enable(PropertySetter):
986
 
        propname = "Enabled"
987
 
        value_to_set = True
988
 
 
989
 
 
990
 
    class Disable(PropertySetter):
991
 
        propname = "Enabled"
992
 
        value_to_set = False
993
 
 
994
 
 
995
 
    class BumpTimeout(PropertySetter):
 
756
    class Enable(Property):
 
757
        propname = "Enabled"
 
758
        value_to_set = dbus.Boolean(True)
 
759
 
 
760
 
 
761
    class Disable(Property):
 
762
        propname = "Enabled"
 
763
        value_to_set = dbus.Boolean(False)
 
764
 
 
765
 
 
766
    class BumpTimeout(Property):
996
767
        propname = "LastCheckedOK"
997
768
        value_to_set = ""
998
769
 
999
770
 
1000
 
    class StartChecker(PropertySetter):
1001
 
        propname = "CheckerRunning"
1002
 
        value_to_set = True
1003
 
 
1004
 
 
1005
 
    class StopChecker(PropertySetter):
1006
 
        propname = "CheckerRunning"
1007
 
        value_to_set = False
1008
 
 
1009
 
 
1010
 
    class ApproveByDefault(PropertySetter):
1011
 
        propname = "ApprovedByDefault"
1012
 
        value_to_set = True
1013
 
 
1014
 
 
1015
 
    class DenyByDefault(PropertySetter):
1016
 
        propname = "ApprovedByDefault"
1017
 
        value_to_set = False
1018
 
 
1019
 
 
1020
 
    class PropertySetterValue(PropertySetter):
1021
 
        """Abstract class for PropertySetter recieving a value as
1022
 
constructor argument instead of a class attribute."""
 
771
    class StartChecker(Property):
 
772
        propname = "CheckerRunning"
 
773
        value_to_set = dbus.Boolean(True)
 
774
 
 
775
 
 
776
    class StopChecker(Property):
 
777
        propname = "CheckerRunning"
 
778
        value_to_set = dbus.Boolean(False)
 
779
 
 
780
 
 
781
    class ApproveByDefault(Property):
 
782
        propname = "ApprovedByDefault"
 
783
        value_to_set = dbus.Boolean(True)
 
784
 
 
785
 
 
786
    class DenyByDefault(Property):
 
787
        propname = "ApprovedByDefault"
 
788
        value_to_set = dbus.Boolean(False)
 
789
 
 
790
 
 
791
    class PropertyValue(Property):
 
792
        "Abstract class for Property recieving a value as argument"
1023
793
        def __init__(self, value):
1024
794
            self.value_to_set = value
1025
795
 
1026
 
        @classmethod
1027
 
        def argparse(cls, argtype):
1028
 
            def cmdtype(arg):
1029
 
                return cls(argtype(arg))
1030
 
            return cmdtype
1031
796
 
1032
 
    class SetChecker(PropertySetterValue):
 
797
    class SetChecker(PropertyValue):
1033
798
        propname = "Checker"
1034
799
 
1035
800
 
1036
 
    class SetHost(PropertySetterValue):
 
801
    class SetHost(PropertyValue):
1037
802
        propname = "Host"
1038
803
 
1039
804
 
1040
 
    class SetSecret(PropertySetterValue):
 
805
    class SetSecret(PropertyValue):
1041
806
        propname = "Secret"
1042
807
 
1043
808
        @property
1051
816
            value.close()
1052
817
 
1053
818
 
1054
 
    class PropertySetterValueMilliseconds(PropertySetterValue):
1055
 
        """Abstract class for PropertySetterValue taking a value
1056
 
argument as a datetime.timedelta() but should store it as
1057
 
milliseconds."""
 
819
    class MillisecondsPropertyValueArgument(PropertyValue):
 
820
        """Abstract class for PropertyValue taking a value argument as
 
821
a datetime.timedelta() but should store it as milliseconds."""
1058
822
 
1059
823
        @property
1060
824
        def value_to_set(self):
1066
830
            self._vts = int(round(value.total_seconds() * 1000))
1067
831
 
1068
832
 
1069
 
    class SetTimeout(PropertySetterValueMilliseconds):
 
833
    class SetTimeout(MillisecondsPropertyValueArgument):
1070
834
        propname = "Timeout"
1071
835
 
1072
836
 
1073
 
    class SetExtendedTimeout(PropertySetterValueMilliseconds):
 
837
    class SetExtendedTimeout(MillisecondsPropertyValueArgument):
1074
838
        propname = "ExtendedTimeout"
1075
839
 
1076
840
 
1077
 
    class SetInterval(PropertySetterValueMilliseconds):
 
841
    class SetInterval(MillisecondsPropertyValueArgument):
1078
842
        propname = "Interval"
1079
843
 
1080
844
 
1081
 
    class SetApprovalDelay(PropertySetterValueMilliseconds):
 
845
    class SetApprovalDelay(MillisecondsPropertyValueArgument):
1082
846
        propname = "ApprovalDelay"
1083
847
 
1084
848
 
1085
 
    class SetApprovalDuration(PropertySetterValueMilliseconds):
 
849
    class SetApprovalDuration(MillisecondsPropertyValueArgument):
1086
850
        propname = "ApprovalDuration"
1087
851
 
1088
852
 
1121
885
                                                     "output"))
1122
886
 
1123
887
 
1124
 
class Unique:
1125
 
    """Class for objects which exist only to be unique objects, since
1126
 
unittest.mock.sentinel only exists in Python 3.3"""
1127
 
 
1128
 
 
1129
888
class Test_string_to_delta(TestCaseWithAssertLogs):
1130
 
    # Just test basic RFC 3339 functionality here, the doc string for
1131
 
    # rfc3339_duration_to_delta() already has more comprehensive
1132
 
    # tests, which are run by doctest.
1133
 
 
1134
 
    def test_rfc3339_zero_seconds(self):
1135
 
        self.assertEqual(datetime.timedelta(),
1136
 
                         string_to_delta("PT0S"))
1137
 
 
1138
 
    def test_rfc3339_zero_days(self):
1139
 
        self.assertEqual(datetime.timedelta(), string_to_delta("P0D"))
1140
 
 
1141
 
    def test_rfc3339_one_second(self):
1142
 
        self.assertEqual(datetime.timedelta(0, 1),
1143
 
                         string_to_delta("PT1S"))
1144
 
 
1145
 
    def test_rfc3339_two_hours(self):
1146
 
        self.assertEqual(datetime.timedelta(0, 7200),
1147
 
                         string_to_delta("PT2H"))
 
889
    def test_handles_basic_rfc3339(self):
 
890
        self.assertEqual(string_to_delta("PT0S"),
 
891
                         datetime.timedelta())
 
892
        self.assertEqual(string_to_delta("P0D"),
 
893
                         datetime.timedelta())
 
894
        self.assertEqual(string_to_delta("PT1S"),
 
895
                         datetime.timedelta(0, 1))
 
896
        self.assertEqual(string_to_delta("PT2H"),
 
897
                         datetime.timedelta(0, 7200))
1148
898
 
1149
899
    def test_falls_back_to_pre_1_6_1_with_warning(self):
1150
900
        with self.assertLogs(log, logging.WARNING):
1151
901
            value = string_to_delta("2h")
1152
 
        self.assertEqual(datetime.timedelta(0, 7200), value)
 
902
        self.assertEqual(value, datetime.timedelta(0, 7200))
1153
903
 
1154
904
 
1155
905
class Test_check_option_syntax(unittest.TestCase):
1159
909
 
1160
910
    def test_actions_requires_client_or_all(self):
1161
911
        for action, value in self.actions.items():
1162
 
            args = self.actionargs(action, value)
 
912
            options = self.parser.parse_args()
 
913
            setattr(options, action, value)
1163
914
            with self.assertParseError():
1164
 
                self.parse_args(args)
 
915
                self.check_option_syntax(options)
1165
916
 
1166
 
    # This mostly corresponds to the definition from has_commands() in
 
917
    # This mostly corresponds to the definition from has_actions() in
1167
918
    # check_option_syntax()
1168
919
    actions = {
1169
 
        "--enable": None,
1170
 
        "--disable": None,
1171
 
        "--bump-timeout": None,
1172
 
        "--start-checker": None,
1173
 
        "--stop-checker": None,
1174
 
        "--is-enabled": None,
1175
 
        "--remove": None,
1176
 
        "--checker": "x",
1177
 
        "--timeout": "PT0S",
1178
 
        "--extended-timeout": "PT0S",
1179
 
        "--interval": "PT0S",
1180
 
        "--approve-by-default": None,
1181
 
        "--deny-by-default": None,
1182
 
        "--approval-delay": "PT0S",
1183
 
        "--approval-duration": "PT0S",
1184
 
        "--host": "hostname",
1185
 
        "--secret": "/dev/null",
1186
 
        "--approve": None,
1187
 
        "--deny": None,
 
920
        # The actual values set here are not that important, but we do
 
921
        # at least stick to the correct types, even though they are
 
922
        # never used
 
923
        "enable": True,
 
924
        "disable": True,
 
925
        "bump_timeout": True,
 
926
        "start_checker": True,
 
927
        "stop_checker": True,
 
928
        "is_enabled": True,
 
929
        "remove": True,
 
930
        "checker": "x",
 
931
        "timeout": datetime.timedelta(),
 
932
        "extended_timeout": datetime.timedelta(),
 
933
        "interval": datetime.timedelta(),
 
934
        "approved_by_default": True,
 
935
        "approval_delay": datetime.timedelta(),
 
936
        "approval_duration": datetime.timedelta(),
 
937
        "host": "x",
 
938
        "secret": io.BytesIO(b"x"),
 
939
        "approve": True,
 
940
        "deny": True,
1188
941
    }
1189
942
 
1190
 
    @staticmethod
1191
 
    def actionargs(action, value, *args):
1192
 
        if value is not None:
1193
 
            return [action, value] + list(args)
1194
 
        else:
1195
 
            return [action] + list(args)
1196
 
 
1197
943
    @contextlib.contextmanager
1198
944
    def assertParseError(self):
1199
945
        with self.assertRaises(SystemExit) as e:
1202
948
        # Exit code from argparse is guaranteed to be "2".  Reference:
1203
949
        # https://docs.python.org/3/library
1204
950
        # /argparse.html#exiting-methods
1205
 
        self.assertEqual(2, e.exception.code)
1206
 
 
1207
 
    def parse_args(self, args):
1208
 
        options = self.parser.parse_args(args)
1209
 
        check_option_syntax(self.parser, options)
 
951
        self.assertEqual(e.exception.code, 2)
1210
952
 
1211
953
    @staticmethod
1212
954
    @contextlib.contextmanager
1213
955
    def redirect_stderr_to_devnull():
1214
 
        old_stderr = sys.stderr
1215
 
        with contextlib.closing(open(os.devnull, "w")) as null:
1216
 
            sys.stderr = null
1217
 
            try:
1218
 
                yield
1219
 
            finally:
1220
 
                sys.stderr = old_stderr
 
956
        null = os.open(os.path.devnull, os.O_RDWR)
 
957
        stderrcopy = os.dup(sys.stderr.fileno())
 
958
        os.dup2(null, sys.stderr.fileno())
 
959
        os.close(null)
 
960
        try:
 
961
            yield
 
962
        finally:
 
963
            # restore stderr
 
964
            os.dup2(stderrcopy, sys.stderr.fileno())
 
965
            os.close(stderrcopy)
1221
966
 
1222
967
    def check_option_syntax(self, options):
1223
968
        check_option_syntax(self.parser, options)
1224
969
 
1225
970
    def test_actions_all_conflicts_with_verbose(self):
1226
971
        for action, value in self.actions.items():
1227
 
            args = self.actionargs(action, value, "--all",
1228
 
                                   "--verbose")
 
972
            options = self.parser.parse_args()
 
973
            setattr(options, action, value)
 
974
            options.all = True
 
975
            options.verbose = True
1229
976
            with self.assertParseError():
1230
 
                self.parse_args(args)
 
977
                self.check_option_syntax(options)
1231
978
 
1232
979
    def test_actions_with_client_conflicts_with_verbose(self):
1233
980
        for action, value in self.actions.items():
1234
 
            args = self.actionargs(action, value, "--verbose",
1235
 
                                   "client")
 
981
            options = self.parser.parse_args()
 
982
            setattr(options, action, value)
 
983
            options.verbose = True
 
984
            options.client = ["foo"]
1236
985
            with self.assertParseError():
1237
 
                self.parse_args(args)
 
986
                self.check_option_syntax(options)
1238
987
 
1239
988
    def test_dump_json_conflicts_with_verbose(self):
1240
 
        args = ["--dump-json", "--verbose"]
 
989
        options = self.parser.parse_args()
 
990
        options.dump_json = True
 
991
        options.verbose = True
1241
992
        with self.assertParseError():
1242
 
            self.parse_args(args)
 
993
            self.check_option_syntax(options)
1243
994
 
1244
995
    def test_dump_json_conflicts_with_action(self):
1245
996
        for action, value in self.actions.items():
1246
 
            args = self.actionargs(action, value, "--dump-json")
 
997
            options = self.parser.parse_args()
 
998
            setattr(options, action, value)
 
999
            options.dump_json = True
1247
1000
            with self.assertParseError():
1248
 
                self.parse_args(args)
 
1001
                self.check_option_syntax(options)
1249
1002
 
1250
1003
    def test_all_can_not_be_alone(self):
1251
 
        args = ["--all"]
 
1004
        options = self.parser.parse_args()
 
1005
        options.all = True
1252
1006
        with self.assertParseError():
1253
 
            self.parse_args(args)
 
1007
            self.check_option_syntax(options)
1254
1008
 
1255
1009
    def test_all_is_ok_with_any_action(self):
1256
1010
        for action, value in self.actions.items():
1257
 
            args = self.actionargs(action, value, "--all")
1258
 
            self.parse_args(args)
 
1011
            options = self.parser.parse_args()
 
1012
            setattr(options, action, value)
 
1013
            options.all = True
 
1014
            self.check_option_syntax(options)
1259
1015
 
1260
1016
    def test_any_action_is_ok_with_one_client(self):
1261
1017
        for action, value in self.actions.items():
1262
 
            args = self.actionargs(action, value, "client")
1263
 
            self.parse_args(args)
 
1018
            options = self.parser.parse_args()
 
1019
            setattr(options, action, value)
 
1020
            options.client = ["foo"]
 
1021
            self.check_option_syntax(options)
1264
1022
 
1265
1023
    def test_one_client_with_all_actions_except_is_enabled(self):
 
1024
        options = self.parser.parse_args()
1266
1025
        for action, value in self.actions.items():
1267
 
            if action == "--is-enabled":
 
1026
            if action == "is_enabled":
1268
1027
                continue
1269
 
            args = self.actionargs(action, value, "client")
1270
 
            self.parse_args(args)
 
1028
            setattr(options, action, value)
 
1029
        options.client = ["foo"]
 
1030
        self.check_option_syntax(options)
1271
1031
 
1272
1032
    def test_two_clients_with_all_actions_except_is_enabled(self):
 
1033
        options = self.parser.parse_args()
1273
1034
        for action, value in self.actions.items():
1274
 
            if action == "--is-enabled":
 
1035
            if action == "is_enabled":
1275
1036
                continue
1276
 
            args = self.actionargs(action, value, "client1",
1277
 
                                   "client2")
1278
 
            self.parse_args(args)
 
1037
            setattr(options, action, value)
 
1038
        options.client = ["foo", "barbar"]
 
1039
        self.check_option_syntax(options)
1279
1040
 
1280
1041
    def test_two_clients_are_ok_with_actions_except_is_enabled(self):
1281
1042
        for action, value in self.actions.items():
1282
 
            if action == "--is-enabled":
 
1043
            if action == "is_enabled":
1283
1044
                continue
1284
 
            args = self.actionargs(action, value, "client1",
1285
 
                                   "client2")
1286
 
            self.parse_args(args)
 
1045
            options = self.parser.parse_args()
 
1046
            setattr(options, action, value)
 
1047
            options.client = ["foo", "barbar"]
 
1048
            self.check_option_syntax(options)
1287
1049
 
1288
1050
    def test_is_enabled_fails_without_client(self):
1289
 
        args = ["--is-enabled"]
 
1051
        options = self.parser.parse_args()
 
1052
        options.is_enabled = True
1290
1053
        with self.assertParseError():
1291
 
            self.parse_args(args)
 
1054
            self.check_option_syntax(options)
1292
1055
 
1293
1056
    def test_is_enabled_fails_with_two_clients(self):
1294
 
        args = ["--is-enabled", "client1", "client2"]
 
1057
        options = self.parser.parse_args()
 
1058
        options.is_enabled = True
 
1059
        options.client = ["foo", "barbar"]
1295
1060
        with self.assertParseError():
1296
 
            self.parse_args(args)
 
1061
            self.check_option_syntax(options)
1297
1062
 
1298
1063
    def test_remove_can_only_be_combined_with_action_deny(self):
1299
1064
        for action, value in self.actions.items():
1300
 
            if action in {"--remove", "--deny"}:
 
1065
            if action in {"remove", "deny"}:
1301
1066
                continue
1302
 
            args = self.actionargs(action, value, "--all",
1303
 
                                   "--remove")
 
1067
            options = self.parser.parse_args()
 
1068
            setattr(options, action, value)
 
1069
            options.all = True
 
1070
            options.remove = True
1304
1071
            with self.assertParseError():
1305
 
                self.parse_args(args)
1306
 
 
1307
 
 
1308
 
class Test_dbus_exceptions(unittest.TestCase):
1309
 
 
1310
 
    def test_dbus_ConnectFailed_is_Error(self):
1311
 
        with self.assertRaises(dbus.Error):
1312
 
            raise dbus.ConnectFailed()
1313
 
 
1314
 
 
1315
 
class Test_dbus_MandosBus(unittest.TestCase):
1316
 
 
1317
 
    class MockMandosBus(dbus.MandosBus):
1318
 
        def __init__(self):
1319
 
            self._name = "se.recompile.Mandos"
1320
 
            self._server_path = "/"
1321
 
            self._server_interface = "se.recompile.Mandos"
1322
 
            self._client_interface = "se.recompile.Mandos.Client"
1323
 
            self.calls = []
1324
 
            self.call_method_return = Unique()
1325
 
 
1326
 
        def call_method(self, methodname, busname, objectpath,
1327
 
                        interface, *args):
1328
 
            self.calls.append((methodname, busname, objectpath,
1329
 
                               interface, args))
1330
 
            return self.call_method_return
1331
 
 
1332
 
    def setUp(self):
1333
 
        self.bus = self.MockMandosBus()
1334
 
 
1335
 
    def test_set_client_property(self):
1336
 
        self.bus.set_client_property("objectpath", "key", "value")
1337
 
        expected_call = ("Set", self.bus._name, "objectpath",
1338
 
                         "org.freedesktop.DBus.Properties",
1339
 
                         (self.bus._client_interface, "key", "value"))
1340
 
        self.assertIn(expected_call, self.bus.calls)
1341
 
 
1342
 
    def test_call_client_method(self):
1343
 
        ret = self.bus.call_client_method("objectpath", "methodname")
1344
 
        self.assertIs(self.bus.call_method_return, ret)
1345
 
        expected_call = ("methodname", self.bus._name, "objectpath",
1346
 
                         self.bus._client_interface, ())
1347
 
        self.assertIn(expected_call, self.bus.calls)
1348
 
 
1349
 
    def test_call_client_method_with_args(self):
1350
 
        args = (Unique(), Unique())
1351
 
        ret = self.bus.call_client_method("objectpath", "methodname",
1352
 
                                          *args)
1353
 
        self.assertIs(self.bus.call_method_return, ret)
1354
 
        expected_call = ("methodname", self.bus._name, "objectpath",
1355
 
                         self.bus._client_interface,
1356
 
                         (args[0], args[1]))
1357
 
        self.assertIn(expected_call, self.bus.calls)
1358
 
 
1359
 
    def test_get_clients_and_properties(self):
1360
 
        managed_objects = {
1361
 
            "objectpath": {
1362
 
                self.bus._client_interface: {
1363
 
                    "key": "value",
1364
 
                    "bool": True,
1365
 
                },
1366
 
                "irrelevant_interface": {
1367
 
                    "key": "othervalue",
1368
 
                    "bool": False,
1369
 
                },
1370
 
            },
1371
 
            "other_objectpath": {
1372
 
                "other_irrelevant_interface": {
1373
 
                    "key": "value 3",
1374
 
                    "bool": None,
1375
 
                },
1376
 
            },
1377
 
        }
1378
 
        expected_clients_and_properties = {
1379
 
            "objectpath": {
1380
 
                "key": "value",
1381
 
                "bool": True,
1382
 
            }
1383
 
        }
1384
 
        self.bus.call_method_return = managed_objects
1385
 
        ret = self.bus.get_clients_and_properties()
1386
 
        self.assertDictEqual(expected_clients_and_properties, ret)
1387
 
        expected_call = ("GetManagedObjects", self.bus._name,
1388
 
                         self.bus._server_path,
1389
 
                         "org.freedesktop.DBus.ObjectManager", ())
1390
 
        self.assertIn(expected_call, self.bus.calls)
1391
 
 
1392
 
    def test_call_server_method(self):
1393
 
        ret = self.bus.call_server_method("methodname")
1394
 
        self.assertIs(self.bus.call_method_return, ret)
1395
 
        expected_call = ("methodname", self.bus._name,
1396
 
                         self.bus._server_path,
1397
 
                         self.bus._server_interface, ())
1398
 
        self.assertIn(expected_call, self.bus.calls)
1399
 
 
1400
 
    def test_call_server_method_with_args(self):
1401
 
        args = (Unique(), Unique())
1402
 
        ret = self.bus.call_server_method("methodname", *args)
1403
 
        self.assertIs(self.bus.call_method_return, ret)
1404
 
        expected_call = ("methodname", self.bus._name,
1405
 
                         self.bus._server_path,
1406
 
                         self.bus._server_interface,
1407
 
                         (args[0], args[1]))
1408
 
        self.assertIn(expected_call, self.bus.calls)
1409
 
 
1410
 
 
1411
 
class Test_dbus_python_adapter_SystemBus(TestCaseWithAssertLogs):
1412
 
 
1413
 
    def MockDBusPython_func(self, func):
1414
 
        class mock_dbus_python:
1415
 
            """mock dbus-python module"""
1416
 
            class exceptions:
1417
 
                """Pseudo-namespace"""
1418
 
                class DBusException(Exception):
1419
 
                    pass
1420
 
            class SystemBus:
1421
 
                @staticmethod
1422
 
                def get_object(busname, objectpath):
1423
 
                    DBusObject = collections.namedtuple(
1424
 
                        "DBusObject", ("methodname", "Set"))
1425
 
                    def method(*args, **kwargs):
1426
 
                        self.assertEqual({"dbus_interface":
1427
 
                                          "interface"},
1428
 
                                         kwargs)
1429
 
                        return func(*args)
1430
 
                    def set_property(interface, key, value,
1431
 
                                     dbus_interface=None):
1432
 
                        self.assertEqual(
1433
 
                            "org.freedesktop.DBus.Properties",
1434
 
                            dbus_interface)
1435
 
                        self.assertEqual("Secret", key)
1436
 
                        return func(interface, key, value,
1437
 
                                    dbus_interface=dbus_interface)
1438
 
                    return DBusObject(methodname=method,
1439
 
                                      Set=set_property)
1440
 
            class Boolean:
1441
 
                def __init__(self, value):
1442
 
                    self.value = bool(value)
1443
 
                def __bool__(self):
1444
 
                    return self.value
1445
 
                if sys.version_info.major == 2:
1446
 
                    __nonzero__ = __bool__
1447
 
            class ObjectPath(str):
1448
 
                pass
1449
 
            class Dictionary(dict):
1450
 
                pass
1451
 
            class ByteArray(bytes):
1452
 
                pass
1453
 
        return mock_dbus_python
1454
 
 
1455
 
    def call_method(self, bus, methodname, busname, objectpath,
1456
 
                    interface, *args):
1457
 
        with self.assertLogs(log, logging.DEBUG):
1458
 
            return bus.call_method(methodname, busname, objectpath,
1459
 
                                   interface, *args)
1460
 
 
1461
 
    def test_call_method_returns(self):
1462
 
        expected_method_return = Unique()
1463
 
        method_args = (Unique(), Unique())
1464
 
        def func(*args):
1465
 
            self.assertEqual(len(method_args), len(args))
1466
 
            for marg, arg in zip(method_args, args):
1467
 
                self.assertIs(marg, arg)
1468
 
            return expected_method_return
1469
 
        mock_dbus_python = self.MockDBusPython_func(func)
1470
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1471
 
        ret = self.call_method(bus, "methodname", "busname",
1472
 
                               "objectpath", "interface",
1473
 
                               *method_args)
1474
 
        self.assertIs(ret, expected_method_return)
1475
 
 
1476
 
    def test_call_method_filters_bool_true(self):
1477
 
        def func():
1478
 
            return method_return
1479
 
        mock_dbus_python = self.MockDBusPython_func(func)
1480
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1481
 
        method_return = mock_dbus_python.Boolean(True)
1482
 
        ret = self.call_method(bus, "methodname", "busname",
1483
 
                               "objectpath", "interface")
1484
 
        self.assertTrue(ret)
1485
 
        self.assertNotIsInstance(ret, mock_dbus_python.Boolean)
1486
 
 
1487
 
    def test_call_method_filters_bool_false(self):
1488
 
        def func():
1489
 
            return method_return
1490
 
        mock_dbus_python = self.MockDBusPython_func(func)
1491
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1492
 
        method_return = mock_dbus_python.Boolean(False)
1493
 
        ret = self.call_method(bus, "methodname", "busname",
1494
 
                               "objectpath", "interface")
1495
 
        self.assertFalse(ret)
1496
 
        self.assertNotIsInstance(ret, mock_dbus_python.Boolean)
1497
 
 
1498
 
    def test_call_method_filters_objectpath(self):
1499
 
        def func():
1500
 
            return method_return
1501
 
        mock_dbus_python = self.MockDBusPython_func(func)
1502
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1503
 
        method_return = mock_dbus_python.ObjectPath("objectpath")
1504
 
        ret = self.call_method(bus, "methodname", "busname",
1505
 
                               "objectpath", "interface")
1506
 
        self.assertEqual("objectpath", ret)
1507
 
        self.assertIsNot("objectpath", ret)
1508
 
        self.assertNotIsInstance(ret, mock_dbus_python.ObjectPath)
1509
 
 
1510
 
    def test_call_method_filters_booleans_in_dict(self):
1511
 
        def func():
1512
 
            return method_return
1513
 
        mock_dbus_python = self.MockDBusPython_func(func)
1514
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1515
 
        method_return = mock_dbus_python.Dictionary(
1516
 
        {mock_dbus_python.Boolean(True):
1517
 
         mock_dbus_python.Boolean(False),
1518
 
         mock_dbus_python.Boolean(False):
1519
 
         mock_dbus_python.Boolean(True)})
1520
 
        ret = self.call_method(bus, "methodname", "busname",
1521
 
                               "objectpath", "interface")
1522
 
        expected_method_return = {True: False,
1523
 
                                  False: True}
1524
 
        self.assertEqual(expected_method_return, ret)
1525
 
        self.assertNotIsInstance(ret, mock_dbus_python.Dictionary)
1526
 
 
1527
 
    def test_call_method_filters_objectpaths_in_dict(self):
1528
 
        def func():
1529
 
            return method_return
1530
 
        mock_dbus_python = self.MockDBusPython_func(func)
1531
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1532
 
        method_return = mock_dbus_python.Dictionary(
1533
 
        {mock_dbus_python.ObjectPath("objectpath_key_1"):
1534
 
         mock_dbus_python.ObjectPath("objectpath_value_1"),
1535
 
         mock_dbus_python.ObjectPath("objectpath_key_2"):
1536
 
         mock_dbus_python.ObjectPath("objectpath_value_2")})
1537
 
        ret = self.call_method(bus, "methodname", "busname",
1538
 
                               "objectpath", "interface")
1539
 
        expected_method_return = {str(key): str(value)
1540
 
                                  for key, value in
1541
 
                                  method_return.items()}
1542
 
        self.assertEqual(expected_method_return, ret)
1543
 
        self.assertIsInstance(ret, dict)
1544
 
        self.assertNotIsInstance(ret, mock_dbus_python.Dictionary)
1545
 
 
1546
 
    def test_call_method_filters_dict_in_dict(self):
1547
 
        def func():
1548
 
            return method_return
1549
 
        mock_dbus_python = self.MockDBusPython_func(func)
1550
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1551
 
        method_return = mock_dbus_python.Dictionary(
1552
 
        {"key1": mock_dbus_python.Dictionary({"key11": "value11",
1553
 
                                              "key12": "value12"}),
1554
 
         "key2": mock_dbus_python.Dictionary({"key21": "value21",
1555
 
                                              "key22": "value22"})})
1556
 
        ret = self.call_method(bus, "methodname", "busname",
1557
 
                               "objectpath", "interface")
1558
 
        expected_method_return = {
1559
 
            "key1": {"key11": "value11",
1560
 
                     "key12": "value12"},
1561
 
            "key2": {"key21": "value21",
1562
 
                     "key22": "value22"},
1563
 
        }
1564
 
        self.assertEqual(expected_method_return, ret)
1565
 
        self.assertIsInstance(ret, dict)
1566
 
        self.assertNotIsInstance(ret, mock_dbus_python.Dictionary)
1567
 
        for key, value in ret.items():
1568
 
            self.assertIsInstance(value, dict)
1569
 
            self.assertEqual(expected_method_return[key], value)
1570
 
            self.assertNotIsInstance(value,
1571
 
                                     mock_dbus_python.Dictionary)
1572
 
 
1573
 
    def test_call_method_filters_dict_three_deep(self):
1574
 
        def func():
1575
 
            return method_return
1576
 
        mock_dbus_python = self.MockDBusPython_func(func)
1577
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1578
 
        method_return = mock_dbus_python.Dictionary(
1579
 
            {"key1":
1580
 
             mock_dbus_python.Dictionary(
1581
 
                 {"key2":
1582
 
                  mock_dbus_python.Dictionary(
1583
 
                      {"key3":
1584
 
                       mock_dbus_python.Boolean(True),
1585
 
                       }),
1586
 
                  }),
1587
 
             })
1588
 
        ret = self.call_method(bus, "methodname", "busname",
1589
 
                               "objectpath", "interface")
1590
 
        expected_method_return = {"key1": {"key2": {"key3": True}}}
1591
 
        self.assertEqual(expected_method_return, ret)
1592
 
        self.assertIsInstance(ret, dict)
1593
 
        self.assertNotIsInstance(ret, mock_dbus_python.Dictionary)
1594
 
        self.assertIsInstance(ret["key1"], dict)
1595
 
        self.assertNotIsInstance(ret["key1"],
1596
 
                                 mock_dbus_python.Dictionary)
1597
 
        self.assertIsInstance(ret["key1"]["key2"], dict)
1598
 
        self.assertNotIsInstance(ret["key1"]["key2"],
1599
 
                                 mock_dbus_python.Dictionary)
1600
 
        self.assertTrue(ret["key1"]["key2"]["key3"])
1601
 
        self.assertNotIsInstance(ret["key1"]["key2"]["key3"],
1602
 
                                 mock_dbus_python.Boolean)
1603
 
 
1604
 
    def test_call_method_handles_exception(self):
 
1072
                self.check_option_syntax(options)
 
1073
 
 
1074
 
 
1075
class Test_get_mandos_dbus_object(TestCaseWithAssertLogs):
 
1076
    def test_calls_and_returns_get_object_on_bus(self):
 
1077
        class MockBus(object):
 
1078
            called = False
 
1079
            def get_object(mockbus_self, busname, dbus_path):
 
1080
                # Note that "self" is still the testcase instance,
 
1081
                # this MockBus instance is in "mockbus_self".
 
1082
                self.assertEqual(busname, dbus_busname)
 
1083
                self.assertEqual(dbus_path, server_dbus_path)
 
1084
                mockbus_self.called = True
 
1085
                return mockbus_self
 
1086
 
 
1087
        mockbus = get_mandos_dbus_object(bus=MockBus())
 
1088
        self.assertIsInstance(mockbus, MockBus)
 
1089
        self.assertTrue(mockbus.called)
 
1090
 
 
1091
    def test_logs_and_exits_on_dbus_error(self):
 
1092
        class MockBusFailing(object):
 
1093
            def get_object(self, busname, dbus_path):
 
1094
                raise dbus.exceptions.DBusException("Test")
 
1095
 
 
1096
        with self.assertLogs(log, logging.CRITICAL):
 
1097
            with self.assertRaises(SystemExit) as e:
 
1098
                bus = get_mandos_dbus_object(bus=MockBusFailing())
 
1099
 
 
1100
        if isinstance(e.exception.code, int):
 
1101
            self.assertNotEqual(e.exception.code, 0)
 
1102
        else:
 
1103
            self.assertIsNotNone(e.exception.code)
 
1104
 
 
1105
 
 
1106
class Test_get_managed_objects(TestCaseWithAssertLogs):
 
1107
    def test_calls_and_returns_GetManagedObjects(self):
 
1108
        managed_objects = {"/clients/foo": { "Name": "foo"}}
 
1109
        class MockObjectManager(object):
 
1110
            def GetManagedObjects(self):
 
1111
                return managed_objects
 
1112
        retval = get_managed_objects(MockObjectManager())
 
1113
        self.assertDictEqual(managed_objects, retval)
 
1114
 
 
1115
    def test_logs_and_exits_on_dbus_error(self):
1605
1116
        dbus_logger = logging.getLogger("dbus.proxies")
1606
1117
 
1607
 
        def func():
1608
 
            dbus_logger.error("Test")
1609
 
            raise mock_dbus_python.exceptions.DBusException()
1610
 
 
1611
 
        mock_dbus_python = self.MockDBusPython_func(func)
1612
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
 
1118
        class MockObjectManagerFailing(object):
 
1119
            def GetManagedObjects(self):
 
1120
                dbus_logger.error("Test")
 
1121
                raise dbus.exceptions.DBusException("Test")
1613
1122
 
1614
1123
        class CountingHandler(logging.Handler):
1615
1124
            count = 0
1621
1130
        dbus_logger.addHandler(counting_handler)
1622
1131
 
1623
1132
        try:
1624
 
            with self.assertRaises(dbus.Error) as e:
1625
 
                self.call_method(bus, "methodname", "busname",
1626
 
                                 "objectpath", "interface")
 
1133
            with self.assertLogs(log, logging.CRITICAL) as watcher:
 
1134
                with self.assertRaises(SystemExit) as e:
 
1135
                    get_managed_objects(MockObjectManagerFailing())
1627
1136
        finally:
1628
1137
            dbus_logger.removeFilter(counting_handler)
1629
1138
 
1630
 
        self.assertNotIsInstance(e, dbus.ConnectFailed)
1631
 
 
1632
1139
        # Make sure the dbus logger was suppressed
1633
 
        self.assertEqual(0, counting_handler.count)
1634
 
 
1635
 
    def test_Set_Secret_sends_bytearray(self):
1636
 
        ret = [None]
1637
 
        def func(*args, **kwargs):
1638
 
            ret[0] = (args, kwargs)
1639
 
        mock_dbus_python = self.MockDBusPython_func(func)
1640
 
        bus = dbus_python_adapter.SystemBus(mock_dbus_python)
1641
 
        bus.set_client_property("objectpath", "Secret", "value")
1642
 
        expected_call = (("se.recompile.Mandos.Client", "Secret",
1643
 
                          mock_dbus_python.ByteArray(b"value")),
1644
 
                         {"dbus_interface":
1645
 
                          "org.freedesktop.DBus.Properties"})
1646
 
        self.assertEqual(expected_call, ret[0])
1647
 
        if sys.version_info.major == 2:
1648
 
            self.assertIsInstance(ret[0][0][-1],
1649
 
                                  mock_dbus_python.ByteArray)
1650
 
 
1651
 
    def test_get_object_converts_to_correct_exception(self):
1652
 
        bus = dbus_python_adapter.SystemBus(
1653
 
            self.fake_dbus_python_raises_exception_on_connect)
1654
 
        with self.assertRaises(dbus.ConnectFailed):
1655
 
            self.call_method(bus, "methodname", "busname",
1656
 
                             "objectpath", "interface")
1657
 
 
1658
 
    class fake_dbus_python_raises_exception_on_connect:
1659
 
        """fake dbus-python module"""
1660
 
        class exceptions:
1661
 
            """Pseudo-namespace"""
1662
 
            class DBusException(Exception):
1663
 
                pass
1664
 
 
1665
 
        @classmethod
1666
 
        def SystemBus(cls):
1667
 
            def get_object(busname, objectpath):
1668
 
                raise cls.exceptions.DBusException()
1669
 
            Bus = collections.namedtuple("Bus", ["get_object"])
1670
 
            return Bus(get_object=get_object)
1671
 
 
1672
 
 
1673
 
class Test_dbus_python_adapter_CachingBus(unittest.TestCase):
1674
 
    class mock_dbus_python:
1675
 
        """mock dbus-python modules"""
1676
 
        class SystemBus:
1677
 
            @staticmethod
1678
 
            def get_object(busname, objectpath):
1679
 
                return Unique()
1680
 
 
1681
 
    def setUp(self):
1682
 
        self.bus = dbus_python_adapter.CachingBus(
1683
 
            self.mock_dbus_python)
1684
 
 
1685
 
    def test_returns_distinct_objectpaths(self):
1686
 
        obj1 = self.bus.get_object("busname", "objectpath1")
1687
 
        self.assertIsInstance(obj1, Unique)
1688
 
        obj2 = self.bus.get_object("busname", "objectpath2")
1689
 
        self.assertIsInstance(obj2, Unique)
1690
 
        self.assertIsNot(obj1, obj2)
1691
 
 
1692
 
    def test_returns_distinct_busnames(self):
1693
 
        obj1 = self.bus.get_object("busname1", "objectpath")
1694
 
        self.assertIsInstance(obj1, Unique)
1695
 
        obj2 = self.bus.get_object("busname2", "objectpath")
1696
 
        self.assertIsInstance(obj2, Unique)
1697
 
        self.assertIsNot(obj1, obj2)
1698
 
 
1699
 
    def test_returns_distinct_both(self):
1700
 
        obj1 = self.bus.get_object("busname1", "objectpath")
1701
 
        self.assertIsInstance(obj1, Unique)
1702
 
        obj2 = self.bus.get_object("busname2", "objectpath")
1703
 
        self.assertIsInstance(obj2, Unique)
1704
 
        self.assertIsNot(obj1, obj2)
1705
 
 
1706
 
    def test_returns_same(self):
1707
 
        obj1 = self.bus.get_object("busname", "objectpath")
1708
 
        self.assertIsInstance(obj1, Unique)
1709
 
        obj2 = self.bus.get_object("busname", "objectpath")
1710
 
        self.assertIsInstance(obj2, Unique)
1711
 
        self.assertIs(obj1, obj2)
1712
 
 
1713
 
    def test_returns_same_old(self):
1714
 
        obj1 = self.bus.get_object("busname1", "objectpath1")
1715
 
        self.assertIsInstance(obj1, Unique)
1716
 
        obj2 = self.bus.get_object("busname2", "objectpath2")
1717
 
        self.assertIsInstance(obj2, Unique)
1718
 
        obj1b = self.bus.get_object("busname1", "objectpath1")
1719
 
        self.assertIsInstance(obj1b, Unique)
1720
 
        self.assertIsNot(obj1, obj2)
1721
 
        self.assertIsNot(obj2, obj1b)
1722
 
        self.assertIs(obj1, obj1b)
1723
 
 
1724
 
 
1725
 
class Test_pydbus_adapter_SystemBus(TestCaseWithAssertLogs):
1726
 
 
1727
 
    def Stub_pydbus_func(self, func):
1728
 
        class stub_pydbus:
1729
 
            """stub pydbus module"""
1730
 
            class SystemBus:
1731
 
                @staticmethod
1732
 
                def get(busname, objectpath):
1733
 
                    DBusObject = collections.namedtuple(
1734
 
                        "DBusObject", ("methodname",))
1735
 
                    return {"interface":
1736
 
                            DBusObject(methodname=func)}
1737
 
        return stub_pydbus
1738
 
 
1739
 
    def call_method(self, bus, methodname, busname, objectpath,
1740
 
                    interface, *args):
1741
 
        with self.assertLogs(log, logging.DEBUG):
1742
 
            return bus.call_method(methodname, busname, objectpath,
1743
 
                                   interface, *args)
1744
 
 
1745
 
    def test_call_method_returns(self):
1746
 
        expected_method_return = Unique()
1747
 
        method_args = (Unique(), Unique())
1748
 
        def func(*args):
1749
 
            self.assertEqual(len(method_args), len(args))
1750
 
            for marg, arg in zip(method_args, args):
1751
 
                self.assertIs(marg, arg)
1752
 
            return expected_method_return
1753
 
        stub_pydbus = self.Stub_pydbus_func(func)
1754
 
        bus = pydbus_adapter.SystemBus(stub_pydbus)
1755
 
        ret = self.call_method(bus, "methodname", "busname",
1756
 
                               "objectpath", "interface",
1757
 
                               *method_args)
1758
 
        self.assertIs(ret, expected_method_return)
1759
 
 
1760
 
    def test_call_method_handles_exception(self):
1761
 
        dbus_logger = logging.getLogger("dbus.proxies")
1762
 
 
1763
 
        def func():
1764
 
            raise gi.repository.GLib.Error()
1765
 
 
1766
 
        stub_pydbus = self.Stub_pydbus_func(func)
1767
 
        bus = pydbus_adapter.SystemBus(stub_pydbus)
1768
 
 
1769
 
        with self.assertRaises(dbus.Error) as e:
1770
 
            self.call_method(bus, "methodname", "busname",
1771
 
                             "objectpath", "interface")
1772
 
 
1773
 
        self.assertNotIsInstance(e, dbus.ConnectFailed)
1774
 
 
1775
 
    def test_get_converts_to_correct_exception(self):
1776
 
        bus = pydbus_adapter.SystemBus(
1777
 
            self.fake_pydbus_raises_exception_on_connect)
1778
 
        with self.assertRaises(dbus.ConnectFailed):
1779
 
            self.call_method(bus, "methodname", "busname",
1780
 
                             "objectpath", "interface")
1781
 
 
1782
 
    class fake_pydbus_raises_exception_on_connect:
1783
 
        """fake dbus-python module"""
1784
 
        @classmethod
1785
 
        def SystemBus(cls):
1786
 
            def get(busname, objectpath):
1787
 
                raise gi.repository.GLib.Error()
1788
 
            Bus = collections.namedtuple("Bus", ["get"])
1789
 
            return Bus(get=get)
1790
 
 
1791
 
    def test_set_property_uses_setattr(self):
1792
 
        class Object:
1793
 
            pass
1794
 
        obj = Object()
1795
 
        class pydbus_spy:
1796
 
            class SystemBus:
1797
 
                @staticmethod
1798
 
                def get(busname, objectpath):
1799
 
                    return {"interface": obj}
1800
 
        bus = pydbus_adapter.SystemBus(pydbus_spy)
1801
 
        value = Unique()
1802
 
        bus.set_property("busname", "objectpath", "interface", "key",
1803
 
                         value)
1804
 
        self.assertIs(value, obj.key)
1805
 
 
1806
 
    def test_get_suppresses_xml_deprecation_warning(self):
1807
 
        if sys.version_info.major >= 3:
1808
 
            return
1809
 
        class stub_pydbus_get:
1810
 
            class SystemBus:
1811
 
                @staticmethod
1812
 
                def get(busname, objectpath):
1813
 
                    warnings.warn_explicit(
1814
 
                        "deprecated", DeprecationWarning,
1815
 
                        "xml.etree.ElementTree", 0)
1816
 
        bus = pydbus_adapter.SystemBus(stub_pydbus_get)
1817
 
        with warnings.catch_warnings(record=True) as w:
1818
 
            warnings.simplefilter("always")
1819
 
            bus.get("busname", "objectpath")
1820
 
            self.assertEqual(0, len(w))
1821
 
 
1822
 
 
1823
 
class Test_pydbus_adapter_CachingBus(unittest.TestCase):
1824
 
    class stub_pydbus:
1825
 
        """stub pydbus module"""
1826
 
        class SystemBus:
1827
 
            @staticmethod
1828
 
            def get(busname, objectpath):
1829
 
                return Unique()
1830
 
 
1831
 
    def setUp(self):
1832
 
        self.bus = pydbus_adapter.CachingBus(self.stub_pydbus)
1833
 
 
1834
 
    def test_returns_distinct_objectpaths(self):
1835
 
        obj1 = self.bus.get("busname", "objectpath1")
1836
 
        self.assertIsInstance(obj1, Unique)
1837
 
        obj2 = self.bus.get("busname", "objectpath2")
1838
 
        self.assertIsInstance(obj2, Unique)
1839
 
        self.assertIsNot(obj1, obj2)
1840
 
 
1841
 
    def test_returns_distinct_busnames(self):
1842
 
        obj1 = self.bus.get("busname1", "objectpath")
1843
 
        self.assertIsInstance(obj1, Unique)
1844
 
        obj2 = self.bus.get("busname2", "objectpath")
1845
 
        self.assertIsInstance(obj2, Unique)
1846
 
        self.assertIsNot(obj1, obj2)
1847
 
 
1848
 
    def test_returns_distinct_both(self):
1849
 
        obj1 = self.bus.get("busname1", "objectpath")
1850
 
        self.assertIsInstance(obj1, Unique)
1851
 
        obj2 = self.bus.get("busname2", "objectpath")
1852
 
        self.assertIsInstance(obj2, Unique)
1853
 
        self.assertIsNot(obj1, obj2)
1854
 
 
1855
 
    def test_returns_same(self):
1856
 
        obj1 = self.bus.get("busname", "objectpath")
1857
 
        self.assertIsInstance(obj1, Unique)
1858
 
        obj2 = self.bus.get("busname", "objectpath")
1859
 
        self.assertIsInstance(obj2, Unique)
1860
 
        self.assertIs(obj1, obj2)
1861
 
 
1862
 
    def test_returns_same_old(self):
1863
 
        obj1 = self.bus.get("busname1", "objectpath1")
1864
 
        self.assertIsInstance(obj1, Unique)
1865
 
        obj2 = self.bus.get("busname2", "objectpath2")
1866
 
        self.assertIsInstance(obj2, Unique)
1867
 
        obj1b = self.bus.get("busname1", "objectpath1")
1868
 
        self.assertIsInstance(obj1b, Unique)
1869
 
        self.assertIsNot(obj1, obj2)
1870
 
        self.assertIsNot(obj2, obj1b)
1871
 
        self.assertIs(obj1, obj1b)
1872
 
 
1873
 
 
1874
 
class Test_dbussy_adapter_SystemBus(TestCaseWithAssertLogs):
1875
 
 
1876
 
    class dummy_dbussy:
1877
 
        class DBUS:
1878
 
            class ObjectPath(str):
1879
 
                pass
1880
 
        class DBusError(Exception):
1881
 
            pass
1882
 
 
1883
 
    def fake_ravel_func(self, func):
1884
 
        class fake_ravel:
1885
 
            @staticmethod
1886
 
            def system_bus():
1887
 
                class DBusInterfaceProxy:
1888
 
                    @staticmethod
1889
 
                    def methodname(*args):
1890
 
                        return [func(*args)]
1891
 
                class DBusObject:
1892
 
                    @staticmethod
1893
 
                    def get_interface(interface):
1894
 
                        if interface == "interface":
1895
 
                            return DBusInterfaceProxy()
1896
 
                return {"busname": {"objectpath": DBusObject()}}
1897
 
        return fake_ravel
1898
 
 
1899
 
    def call_method(self, bus, methodname, busname, objectpath,
1900
 
                    interface, *args):
1901
 
        with self.assertLogs(log, logging.DEBUG):
1902
 
            return bus.call_method(methodname, busname, objectpath,
1903
 
                                   interface, *args)
1904
 
 
1905
 
    def test_call_method_returns(self):
1906
 
        expected_method_return = Unique()
1907
 
        method_args = (Unique(), Unique())
1908
 
        def func(*args):
1909
 
            self.assertEqual(len(method_args), len(args))
1910
 
            for marg, arg in zip(method_args, args):
1911
 
                self.assertIs(marg, arg)
1912
 
            return expected_method_return
1913
 
        fake_ravel = self.fake_ravel_func(func)
1914
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
1915
 
        ret = self.call_method(bus, "methodname", "busname",
1916
 
                               "objectpath", "interface",
1917
 
                               *method_args)
1918
 
        self.assertIs(ret, expected_method_return)
1919
 
 
1920
 
    def test_call_method_filters_objectpath(self):
1921
 
        def func():
1922
 
            return method_return
1923
 
        fake_ravel = self.fake_ravel_func(func)
1924
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
1925
 
        method_return = (self.dummy_dbussy.DBUS
1926
 
                         .ObjectPath("objectpath"))
1927
 
        ret = self.call_method(bus, "methodname", "busname",
1928
 
                               "objectpath", "interface")
1929
 
        self.assertEqual("objectpath", ret)
1930
 
        self.assertNotIsInstance(ret,
1931
 
                                 self.dummy_dbussy.DBUS.ObjectPath)
1932
 
 
1933
 
    def test_call_method_filters_objectpaths_in_dict(self):
1934
 
        ObjectPath = self.dummy_dbussy.DBUS.ObjectPath
1935
 
        def func():
1936
 
            return method_return
1937
 
        fake_ravel = self.fake_ravel_func(func)
1938
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
1939
 
        method_return = {
1940
 
            ObjectPath("objectpath_key_1"):
1941
 
            ObjectPath("objectpath_value_1"),
1942
 
            ObjectPath("objectpath_key_2"):
1943
 
            ObjectPath("objectpath_value_2"),
1944
 
        }
1945
 
        ret = self.call_method(bus, "methodname", "busname",
1946
 
                               "objectpath", "interface")
1947
 
        expected_method_return = {str(key): str(value)
1948
 
                                  for key, value in
1949
 
                                  method_return.items()}
1950
 
        for key, value in ret.items():
1951
 
            self.assertNotIsInstance(key, ObjectPath)
1952
 
            self.assertNotIsInstance(value, ObjectPath)
1953
 
        self.assertEqual(expected_method_return, ret)
1954
 
        self.assertIsInstance(ret, dict)
1955
 
 
1956
 
    def test_call_method_filters_objectpaths_in_dict_in_dict(self):
1957
 
        ObjectPath = self.dummy_dbussy.DBUS.ObjectPath
1958
 
        def func():
1959
 
            return method_return
1960
 
        fake_ravel = self.fake_ravel_func(func)
1961
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
1962
 
        method_return = {
1963
 
            ObjectPath("key1"): {
1964
 
                ObjectPath("key11"): ObjectPath("value11"),
1965
 
                ObjectPath("key12"): ObjectPath("value12"),
1966
 
            },
1967
 
            ObjectPath("key2"): {
1968
 
                ObjectPath("key21"): ObjectPath("value21"),
1969
 
                ObjectPath("key22"): ObjectPath("value22"),
1970
 
            },
1971
 
        }
1972
 
        ret = self.call_method(bus, "methodname", "busname",
1973
 
                               "objectpath", "interface")
1974
 
        expected_method_return = {
1975
 
            "key1": {"key11": "value11",
1976
 
                     "key12": "value12"},
1977
 
            "key2": {"key21": "value21",
1978
 
                     "key22": "value22"},
1979
 
        }
1980
 
        self.assertEqual(expected_method_return, ret)
1981
 
        for key, value in ret.items():
1982
 
            self.assertIsInstance(value, dict)
1983
 
            self.assertEqual(expected_method_return[key], value)
1984
 
            self.assertNotIsInstance(key, ObjectPath)
1985
 
            for inner_key, inner_value in value.items():
1986
 
                self.assertIsInstance(value, dict)
1987
 
                self.assertEqual(
1988
 
                    expected_method_return[key][inner_key],
1989
 
                    inner_value)
1990
 
                self.assertNotIsInstance(key, ObjectPath)
1991
 
 
1992
 
    def test_call_method_filters_objectpaths_in_dict_three_deep(self):
1993
 
        ObjectPath = self.dummy_dbussy.DBUS.ObjectPath
1994
 
        def func():
1995
 
            return method_return
1996
 
        fake_ravel = self.fake_ravel_func(func)
1997
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
1998
 
        method_return = {
1999
 
            ObjectPath("key1"): {
2000
 
                ObjectPath("key2"): {
2001
 
                    ObjectPath("key3"): ObjectPath("value"),
2002
 
                },
2003
 
            },
2004
 
        }
2005
 
        ret = self.call_method(bus, "methodname", "busname",
2006
 
                               "objectpath", "interface")
2007
 
        expected_method_return = {"key1": {"key2": {"key3": "value"}}}
2008
 
        self.assertEqual(expected_method_return, ret)
2009
 
        self.assertIsInstance(ret, dict)
2010
 
        self.assertNotIsInstance(next(iter(ret.keys())), ObjectPath)
2011
 
        self.assertIsInstance(ret["key1"], dict)
2012
 
        self.assertNotIsInstance(next(iter(ret["key1"].keys())),
2013
 
                                 ObjectPath)
2014
 
        self.assertIsInstance(ret["key1"]["key2"], dict)
2015
 
        self.assertNotIsInstance(
2016
 
            next(iter(ret["key1"]["key2"].keys())),
2017
 
            ObjectPath)
2018
 
        self.assertEqual("value", ret["key1"]["key2"]["key3"])
2019
 
        self.assertNotIsInstance(ret["key1"]["key2"]["key3"],
2020
 
                                 self.dummy_dbussy.DBUS.ObjectPath)
2021
 
 
2022
 
    def test_call_method_handles_exception(self):
2023
 
        def func():
2024
 
            raise self.dummy_dbussy.DBusError()
2025
 
 
2026
 
        fake_ravel = self.fake_ravel_func(func)
2027
 
        bus = dbussy_adapter.SystemBus(self.dummy_dbussy, fake_ravel)
2028
 
 
2029
 
        with self.assertRaises(dbus.Error) as e:
2030
 
            self.call_method(bus, "methodname", "busname",
2031
 
                             "objectpath", "interface")
2032
 
 
2033
 
        self.assertNotIsInstance(e.exception, dbus.ConnectFailed)
2034
 
 
2035
 
    def test_get_object_converts_to_correct_exception(self):
2036
 
        class fake_ravel_raises_exception_on_connect:
2037
 
            @staticmethod
2038
 
            def system_bus():
2039
 
                class Bus:
2040
 
                    @staticmethod
2041
 
                    def __getitem__(key):
2042
 
                        if key == "objectpath":
2043
 
                            raise self.dummy_dbussy.DBusError()
2044
 
                        raise Exception(key)
2045
 
                return {"busname": Bus()}
2046
 
        def func():
2047
 
            raise self.dummy_dbussy.DBusError()
2048
 
        bus = dbussy_adapter.SystemBus(
2049
 
            self.dummy_dbussy,
2050
 
            fake_ravel_raises_exception_on_connect)
2051
 
        with self.assertRaises(dbus.ConnectFailed):
2052
 
            self.call_method(bus, "methodname", "busname",
2053
 
                             "objectpath", "interface")
 
1140
        self.assertEqual(counting_handler.count, 0)
 
1141
 
 
1142
        # Test that the dbus_logger still works
 
1143
        with self.assertLogs(dbus_logger, logging.ERROR):
 
1144
            dbus_logger.error("Test")
 
1145
 
 
1146
        if isinstance(e.exception.code, int):
 
1147
            self.assertNotEqual(e.exception.code, 0)
 
1148
        else:
 
1149
            self.assertIsNotNone(e.exception.code)
2054
1150
 
2055
1151
 
2056
1152
class Test_commands_from_options(unittest.TestCase):
2057
 
 
2058
1153
    def setUp(self):
2059
1154
        self.parser = argparse.ArgumentParser()
2060
1155
        add_command_line_options(self.parser)
2061
1156
 
2062
1157
    def test_is_enabled(self):
2063
 
        self.assert_command_from_args(["--is-enabled", "client"],
 
1158
        self.assert_command_from_args(["--is-enabled", "foo"],
2064
1159
                                      command.IsEnabled)
2065
1160
 
2066
 
    def assert_command_from_args(self, args, command_cls, length=1,
2067
 
                                 clients=None, **cmd_attrs):
 
1161
    def assert_command_from_args(self, args, command_cls,
 
1162
                                 **cmd_attrs):
2068
1163
        """Assert that parsing ARGS should result in an instance of
2069
1164
COMMAND_CLS with (optionally) all supplied attributes (CMD_ATTRS)."""
2070
1165
        options = self.parser.parse_args(args)
2071
1166
        check_option_syntax(self.parser, options)
2072
1167
        commands = commands_from_options(options)
2073
 
        self.assertEqual(length, len(commands))
2074
 
        for command in commands:
2075
 
            if isinstance(command, command_cls):
2076
 
                break
2077
 
        else:
2078
 
            self.assertIsInstance(command, command_cls)
2079
 
        if clients is not None:
2080
 
            self.assertEqual(clients, options.client)
 
1168
        self.assertEqual(len(commands), 1)
 
1169
        command = commands[0]
 
1170
        self.assertIsInstance(command, command_cls)
2081
1171
        for key, value in cmd_attrs.items():
2082
 
            self.assertEqual(value, getattr(command, key))
2083
 
 
2084
 
    def assert_commands_from_args(self, args, commands, clients=None):
2085
 
        for cmd in commands:
2086
 
            self.assert_command_from_args(args, cmd,
2087
 
                                          length=len(commands),
2088
 
                                          clients=clients)
 
1172
            self.assertEqual(getattr(command, key), value)
2089
1173
 
2090
1174
    def test_is_enabled_short(self):
2091
 
        self.assert_command_from_args(["-V", "client"],
 
1175
        self.assert_command_from_args(["-V", "foo"],
2092
1176
                                      command.IsEnabled)
2093
1177
 
2094
1178
    def test_approve(self):
2095
 
        self.assert_command_from_args(["--approve", "client"],
 
1179
        self.assert_command_from_args(["--approve", "foo"],
2096
1180
                                      command.Approve)
2097
1181
 
2098
1182
    def test_approve_short(self):
2099
 
        self.assert_command_from_args(["-A", "client"],
2100
 
                                      command.Approve)
 
1183
        self.assert_command_from_args(["-A", "foo"], command.Approve)
2101
1184
 
2102
1185
    def test_deny(self):
2103
 
        self.assert_command_from_args(["--deny", "client"],
2104
 
                                      command.Deny)
 
1186
        self.assert_command_from_args(["--deny", "foo"], command.Deny)
2105
1187
 
2106
1188
    def test_deny_short(self):
2107
 
        self.assert_command_from_args(["-D", "client"], command.Deny)
 
1189
        self.assert_command_from_args(["-D", "foo"], command.Deny)
2108
1190
 
2109
1191
    def test_remove(self):
2110
 
        self.assert_command_from_args(["--remove", "client"],
 
1192
        self.assert_command_from_args(["--remove", "foo"],
2111
1193
                                      command.Remove)
2112
1194
 
2113
1195
    def test_deny_before_remove(self):
2114
1196
        options = self.parser.parse_args(["--deny", "--remove",
2115
 
                                          "client"])
 
1197
                                          "foo"])
2116
1198
        check_option_syntax(self.parser, options)
2117
1199
        commands = commands_from_options(options)
2118
 
        self.assertEqual(2, len(commands))
 
1200
        self.assertEqual(len(commands), 2)
2119
1201
        self.assertIsInstance(commands[0], command.Deny)
2120
1202
        self.assertIsInstance(commands[1], command.Remove)
2121
1203
 
2124
1206
                                          "--all"])
2125
1207
        check_option_syntax(self.parser, options)
2126
1208
        commands = commands_from_options(options)
2127
 
        self.assertEqual(2, len(commands))
 
1209
        self.assertEqual(len(commands), 2)
2128
1210
        self.assertIsInstance(commands[0], command.Deny)
2129
1211
        self.assertIsInstance(commands[1], command.Remove)
2130
1212
 
2131
1213
    def test_remove_short(self):
2132
 
        self.assert_command_from_args(["-r", "client"],
2133
 
                                      command.Remove)
 
1214
        self.assert_command_from_args(["-r", "foo"], command.Remove)
2134
1215
 
2135
1216
    def test_dump_json(self):
2136
1217
        self.assert_command_from_args(["--dump-json"],
2137
1218
                                      command.DumpJSON)
2138
1219
 
2139
1220
    def test_enable(self):
2140
 
        self.assert_command_from_args(["--enable", "client"],
 
1221
        self.assert_command_from_args(["--enable", "foo"],
2141
1222
                                      command.Enable)
2142
1223
 
2143
1224
    def test_enable_short(self):
2144
 
        self.assert_command_from_args(["-e", "client"],
2145
 
                                      command.Enable)
 
1225
        self.assert_command_from_args(["-e", "foo"], command.Enable)
2146
1226
 
2147
1227
    def test_disable(self):
2148
 
        self.assert_command_from_args(["--disable", "client"],
 
1228
        self.assert_command_from_args(["--disable", "foo"],
2149
1229
                                      command.Disable)
2150
1230
 
2151
1231
    def test_disable_short(self):
2152
 
        self.assert_command_from_args(["-d", "client"],
2153
 
                                      command.Disable)
 
1232
        self.assert_command_from_args(["-d", "foo"], command.Disable)
2154
1233
 
2155
1234
    def test_bump_timeout(self):
2156
 
        self.assert_command_from_args(["--bump-timeout", "client"],
 
1235
        self.assert_command_from_args(["--bump-timeout", "foo"],
2157
1236
                                      command.BumpTimeout)
2158
1237
 
2159
1238
    def test_bump_timeout_short(self):
2160
 
        self.assert_command_from_args(["-b", "client"],
 
1239
        self.assert_command_from_args(["-b", "foo"],
2161
1240
                                      command.BumpTimeout)
2162
1241
 
2163
1242
    def test_start_checker(self):
2164
 
        self.assert_command_from_args(["--start-checker", "client"],
 
1243
        self.assert_command_from_args(["--start-checker", "foo"],
2165
1244
                                      command.StartChecker)
2166
1245
 
2167
1246
    def test_stop_checker(self):
2168
 
        self.assert_command_from_args(["--stop-checker", "client"],
 
1247
        self.assert_command_from_args(["--stop-checker", "foo"],
2169
1248
                                      command.StopChecker)
2170
1249
 
2171
1250
    def test_approve_by_default(self):
2172
 
        self.assert_command_from_args(["--approve-by-default",
2173
 
                                       "client"],
 
1251
        self.assert_command_from_args(["--approve-by-default", "foo"],
2174
1252
                                      command.ApproveByDefault)
2175
1253
 
2176
1254
    def test_deny_by_default(self):
2177
 
        self.assert_command_from_args(["--deny-by-default", "client"],
 
1255
        self.assert_command_from_args(["--deny-by-default", "foo"],
2178
1256
                                      command.DenyByDefault)
2179
1257
 
2180
1258
    def test_checker(self):
2181
 
        self.assert_command_from_args(["--checker", ":", "client"],
 
1259
        self.assert_command_from_args(["--checker", ":", "foo"],
2182
1260
                                      command.SetChecker,
2183
1261
                                      value_to_set=":")
2184
1262
 
2185
1263
    def test_checker_empty(self):
2186
 
        self.assert_command_from_args(["--checker", "", "client"],
 
1264
        self.assert_command_from_args(["--checker", "", "foo"],
2187
1265
                                      command.SetChecker,
2188
1266
                                      value_to_set="")
2189
1267
 
2190
1268
    def test_checker_short(self):
2191
 
        self.assert_command_from_args(["-c", ":", "client"],
 
1269
        self.assert_command_from_args(["-c", ":", "foo"],
2192
1270
                                      command.SetChecker,
2193
1271
                                      value_to_set=":")
2194
1272
 
2195
1273
    def test_host(self):
2196
 
        self.assert_command_from_args(
2197
 
            ["--host", "client.example.org", "client"],
2198
 
            command.SetHost, value_to_set="client.example.org")
 
1274
        self.assert_command_from_args(["--host", "foo.example.org",
 
1275
                                       "foo"], command.SetHost,
 
1276
                                      value_to_set="foo.example.org")
2199
1277
 
2200
1278
    def test_host_short(self):
2201
 
        self.assert_command_from_args(
2202
 
            ["-H", "client.example.org", "client"], command.SetHost,
2203
 
            value_to_set="client.example.org")
 
1279
        self.assert_command_from_args(["-H", "foo.example.org",
 
1280
                                       "foo"], command.SetHost,
 
1281
                                      value_to_set="foo.example.org")
2204
1282
 
2205
1283
    def test_secret_devnull(self):
2206
1284
        self.assert_command_from_args(["--secret", os.path.devnull,
2207
 
                                       "client"], command.SetSecret,
 
1285
                                       "foo"], command.SetSecret,
2208
1286
                                      value_to_set=b"")
2209
1287
 
2210
1288
    def test_secret_tempfile(self):
2213
1291
            f.write(value)
2214
1292
            f.seek(0)
2215
1293
            self.assert_command_from_args(["--secret", f.name,
2216
 
                                           "client"],
2217
 
                                          command.SetSecret,
 
1294
                                           "foo"], command.SetSecret,
2218
1295
                                          value_to_set=value)
2219
1296
 
2220
1297
    def test_secret_devnull_short(self):
2221
 
        self.assert_command_from_args(["-s", os.path.devnull,
2222
 
                                       "client"], command.SetSecret,
 
1298
        self.assert_command_from_args(["-s", os.path.devnull, "foo"],
 
1299
                                      command.SetSecret,
2223
1300
                                      value_to_set=b"")
2224
1301
 
2225
1302
    def test_secret_tempfile_short(self):
2227
1304
            value = b"secret\0xyzzy\nbar"
2228
1305
            f.write(value)
2229
1306
            f.seek(0)
2230
 
            self.assert_command_from_args(["-s", f.name, "client"],
 
1307
            self.assert_command_from_args(["-s", f.name, "foo"],
2231
1308
                                          command.SetSecret,
2232
1309
                                          value_to_set=value)
2233
1310
 
2234
1311
    def test_timeout(self):
2235
 
        self.assert_command_from_args(["--timeout", "PT5M", "client"],
 
1312
        self.assert_command_from_args(["--timeout", "PT5M", "foo"],
2236
1313
                                      command.SetTimeout,
2237
1314
                                      value_to_set=300000)
2238
1315
 
2239
1316
    def test_timeout_short(self):
2240
 
        self.assert_command_from_args(["-t", "PT5M", "client"],
 
1317
        self.assert_command_from_args(["-t", "PT5M", "foo"],
2241
1318
                                      command.SetTimeout,
2242
1319
                                      value_to_set=300000)
2243
1320
 
2244
1321
    def test_extended_timeout(self):
2245
1322
        self.assert_command_from_args(["--extended-timeout", "PT15M",
2246
 
                                       "client"],
 
1323
                                       "foo"],
2247
1324
                                      command.SetExtendedTimeout,
2248
1325
                                      value_to_set=900000)
2249
1326
 
2250
1327
    def test_interval(self):
2251
 
        self.assert_command_from_args(["--interval", "PT2M",
2252
 
                                       "client"], command.SetInterval,
 
1328
        self.assert_command_from_args(["--interval", "PT2M", "foo"],
 
1329
                                      command.SetInterval,
2253
1330
                                      value_to_set=120000)
2254
1331
 
2255
1332
    def test_interval_short(self):
2256
 
        self.assert_command_from_args(["-i", "PT2M", "client"],
 
1333
        self.assert_command_from_args(["-i", "PT2M", "foo"],
2257
1334
                                      command.SetInterval,
2258
1335
                                      value_to_set=120000)
2259
1336
 
2260
1337
    def test_approval_delay(self):
2261
1338
        self.assert_command_from_args(["--approval-delay", "PT30S",
2262
 
                                       "client"],
 
1339
                                       "foo"],
2263
1340
                                      command.SetApprovalDelay,
2264
1341
                                      value_to_set=30000)
2265
1342
 
2266
1343
    def test_approval_duration(self):
2267
1344
        self.assert_command_from_args(["--approval-duration", "PT1S",
2268
 
                                       "client"],
 
1345
                                       "foo"],
2269
1346
                                      command.SetApprovalDuration,
2270
1347
                                      value_to_set=1000)
2271
1348
 
2283
1360
                                      verbose=True)
2284
1361
 
2285
1362
 
2286
 
    def test_manual_page_example_1(self):
2287
 
        self.assert_command_from_args("",
2288
 
                                      command.PrintTable,
2289
 
                                      clients=[],
2290
 
                                      verbose=False)
2291
 
 
2292
 
    def test_manual_page_example_2(self):
2293
 
        self.assert_command_from_args(
2294
 
            "--verbose foo1.example.org foo2.example.org".split(),
2295
 
            command.PrintTable, clients=["foo1.example.org",
2296
 
                                         "foo2.example.org"],
2297
 
            verbose=True)
2298
 
 
2299
 
    def test_manual_page_example_3(self):
2300
 
        self.assert_command_from_args("--enable --all".split(),
2301
 
                                      command.Enable,
2302
 
                                      clients=[])
2303
 
 
2304
 
    def test_manual_page_example_4(self):
2305
 
        self.assert_commands_from_args(
2306
 
            ("--timeout=PT5M --interval=PT1M foo1.example.org"
2307
 
             " foo2.example.org").split(),
2308
 
            [command.SetTimeout, command.SetInterval],
2309
 
            clients=["foo1.example.org", "foo2.example.org"])
2310
 
 
2311
 
    def test_manual_page_example_5(self):
2312
 
        self.assert_command_from_args("--approve --all".split(),
2313
 
                                      command.Approve,
2314
 
                                      clients=[])
2315
 
 
2316
 
 
2317
1363
class TestCommand(unittest.TestCase):
2318
1364
    """Abstract class for tests of command classes"""
2319
1365
 
2320
 
    class FakeMandosBus(dbus.MandosBus):
2321
 
        def __init__(self, testcase):
2322
 
            self.client_properties = {
2323
 
                "Name": "foo",
2324
 
                "KeyID": ("92ed150794387c03ce684574b1139a65"
2325
 
                          "94a34f895daaaf09fd8ea90a27cddb12"),
2326
 
                "Secret": b"secret",
2327
 
                "Host": "foo.example.org",
2328
 
                "Enabled": True,
2329
 
                "Timeout": 300000,
2330
 
                "LastCheckedOK": "2019-02-03T00:00:00",
2331
 
                "Created": "2019-01-02T00:00:00",
2332
 
                "Interval": 120000,
2333
 
                "Fingerprint": ("778827225BA7DE539C5A"
2334
 
                                "7CFA59CFF7CDBD9A5920"),
2335
 
                "CheckerRunning": False,
2336
 
                "LastEnabled": "2019-01-03T00:00:00",
2337
 
                "ApprovalPending": False,
2338
 
                "ApprovedByDefault": True,
2339
 
                "LastApprovalRequest": "",
2340
 
                "ApprovalDelay": 0,
2341
 
                "ApprovalDuration": 1000,
2342
 
                "Checker": "fping -q -- %(host)s",
2343
 
                "ExtendedTimeout": 900000,
2344
 
                "Expires": "2019-02-04T00:00:00",
2345
 
                "LastCheckerStatus": 0,
2346
 
            }
2347
 
            self.other_client_properties = {
2348
 
                "Name": "barbar",
2349
 
                "KeyID": ("0558568eedd67d622f5c83b35a115f79"
2350
 
                          "6ab612cff5ad227247e46c2b020f441c"),
2351
 
                "Secret": b"secretbar",
2352
 
                "Host": "192.0.2.3",
2353
 
                "Enabled": True,
2354
 
                "Timeout": 300000,
2355
 
                "LastCheckedOK": "2019-02-04T00:00:00",
2356
 
                "Created": "2019-01-03T00:00:00",
2357
 
                "Interval": 120000,
2358
 
                "Fingerprint": ("3E393AEAEFB84C7E89E2"
2359
 
                                "F547B3A107558FCA3A27"),
2360
 
                "CheckerRunning": True,
2361
 
                "LastEnabled": "2019-01-04T00:00:00",
2362
 
                "ApprovalPending": False,
2363
 
                "ApprovedByDefault": False,
2364
 
                "LastApprovalRequest": "2019-01-03T00:00:00",
2365
 
                "ApprovalDelay": 30000,
2366
 
                "ApprovalDuration": 93785000,
2367
 
                "Checker": ":",
2368
 
                "ExtendedTimeout": 900000,
2369
 
                "Expires": "2019-02-05T00:00:00",
2370
 
                "LastCheckerStatus": -2,
2371
 
            }
2372
 
            self.clients =  collections.OrderedDict(
2373
 
                [
2374
 
                    ("client_objectpath", self.client_properties),
2375
 
                    ("other_client_objectpath",
2376
 
                     self.other_client_properties),
2377
 
                ])
2378
 
            self.one_client = {"client_objectpath":
2379
 
                               self.client_properties}
2380
 
            self.testcase = testcase
2381
 
            self.calls = []
2382
 
 
2383
 
        def call_method(self, methodname, busname, objectpath,
2384
 
                        interface, *args):
2385
 
            self.testcase.assertEqual("se.recompile.Mandos", busname)
2386
 
            self.calls.append((methodname, busname, objectpath,
2387
 
                               interface, args))
2388
 
            if interface == "org.freedesktop.DBus.Properties":
2389
 
                if methodname == "Set":
2390
 
                    self.testcase.assertEqual(3, len(args))
2391
 
                    interface, key, value = args
2392
 
                    self.testcase.assertEqual(
2393
 
                        "se.recompile.Mandos.Client", interface)
2394
 
                    self.clients[objectpath][key] = value
2395
 
                    return
2396
 
            elif interface == "se.recompile.Mandos":
2397
 
                self.testcase.assertEqual("RemoveClient", methodname)
2398
 
                self.testcase.assertEqual(1, len(args))
2399
 
                clientpath = args[0]
2400
 
                del self.clients[clientpath]
2401
 
                return
2402
 
            elif interface == "se.recompile.Mandos.Client":
2403
 
                if methodname == "Approve":
2404
 
                    self.testcase.assertEqual(1, len(args))
2405
 
                    return
2406
 
            raise ValueError()
2407
 
 
2408
1366
    def setUp(self):
2409
 
        self.bus = self.FakeMandosBus(self)
 
1367
        testcase = self
 
1368
        class MockClient(object):
 
1369
            def __init__(self, name, **attributes):
 
1370
                self.__dbus_object_path__ = "/clients/{}".format(name)
 
1371
                self.attributes = attributes
 
1372
                self.attributes["Name"] = name
 
1373
                self.calls = []
 
1374
            def Set(self, interface, propname, value, dbus_interface):
 
1375
                testcase.assertEqual(interface, client_dbus_interface)
 
1376
                testcase.assertEqual(dbus_interface,
 
1377
                                     dbus.PROPERTIES_IFACE)
 
1378
                self.attributes[propname] = value
 
1379
            def Get(self, interface, propname, dbus_interface):
 
1380
                testcase.assertEqual(interface, client_dbus_interface)
 
1381
                testcase.assertEqual(dbus_interface,
 
1382
                                     dbus.PROPERTIES_IFACE)
 
1383
                return self.attributes[propname]
 
1384
            def Approve(self, approve, dbus_interface):
 
1385
                testcase.assertEqual(dbus_interface,
 
1386
                                     client_dbus_interface)
 
1387
                self.calls.append(("Approve", (approve,
 
1388
                                               dbus_interface)))
 
1389
        self.client = MockClient(
 
1390
            "foo",
 
1391
            KeyID=("92ed150794387c03ce684574b1139a65"
 
1392
                   "94a34f895daaaf09fd8ea90a27cddb12"),
 
1393
            Secret=b"secret",
 
1394
            Host="foo.example.org",
 
1395
            Enabled=dbus.Boolean(True),
 
1396
            Timeout=300000,
 
1397
            LastCheckedOK="2019-02-03T00:00:00",
 
1398
            Created="2019-01-02T00:00:00",
 
1399
            Interval=120000,
 
1400
            Fingerprint=("778827225BA7DE539C5A"
 
1401
                         "7CFA59CFF7CDBD9A5920"),
 
1402
            CheckerRunning=dbus.Boolean(False),
 
1403
            LastEnabled="2019-01-03T00:00:00",
 
1404
            ApprovalPending=dbus.Boolean(False),
 
1405
            ApprovedByDefault=dbus.Boolean(True),
 
1406
            LastApprovalRequest="",
 
1407
            ApprovalDelay=0,
 
1408
            ApprovalDuration=1000,
 
1409
            Checker="fping -q -- %(host)s",
 
1410
            ExtendedTimeout=900000,
 
1411
            Expires="2019-02-04T00:00:00",
 
1412
            LastCheckerStatus=0)
 
1413
        self.other_client = MockClient(
 
1414
            "barbar",
 
1415
            KeyID=("0558568eedd67d622f5c83b35a115f79"
 
1416
                   "6ab612cff5ad227247e46c2b020f441c"),
 
1417
            Secret=b"secretbar",
 
1418
            Host="192.0.2.3",
 
1419
            Enabled=dbus.Boolean(True),
 
1420
            Timeout=300000,
 
1421
            LastCheckedOK="2019-02-04T00:00:00",
 
1422
            Created="2019-01-03T00:00:00",
 
1423
            Interval=120000,
 
1424
            Fingerprint=("3E393AEAEFB84C7E89E2"
 
1425
                         "F547B3A107558FCA3A27"),
 
1426
            CheckerRunning=dbus.Boolean(True),
 
1427
            LastEnabled="2019-01-04T00:00:00",
 
1428
            ApprovalPending=dbus.Boolean(False),
 
1429
            ApprovedByDefault=dbus.Boolean(False),
 
1430
            LastApprovalRequest="2019-01-03T00:00:00",
 
1431
            ApprovalDelay=30000,
 
1432
            ApprovalDuration=93785000,
 
1433
            Checker=":",
 
1434
            ExtendedTimeout=900000,
 
1435
            Expires="2019-02-05T00:00:00",
 
1436
            LastCheckerStatus=-2)
 
1437
        self.clients =  collections.OrderedDict(
 
1438
            [
 
1439
                ("/clients/foo", self.client.attributes),
 
1440
                ("/clients/barbar", self.other_client.attributes),
 
1441
            ])
 
1442
        self.one_client = {"/clients/foo": self.client.attributes}
 
1443
 
 
1444
    @property
 
1445
    def bus(self):
 
1446
        class Bus(object):
 
1447
            @staticmethod
 
1448
            def get_object(client_bus_name, path):
 
1449
                self.assertEqual(client_bus_name, dbus_busname)
 
1450
                return {
 
1451
                    # Note: "self" here is the TestCmd instance, not
 
1452
                    # the Bus instance, since this is a static method!
 
1453
                    "/clients/foo": self.client,
 
1454
                    "/clients/barbar": self.other_client,
 
1455
                }[path]
 
1456
        return Bus()
2410
1457
 
2411
1458
 
2412
1459
class TestBaseCommands(TestCommand):
2413
1460
 
2414
 
    def test_IsEnabled_exits_successfully(self):
 
1461
    def test_is_enabled(self):
 
1462
        self.assertTrue(all(command.IsEnabled().is_enabled(client,
 
1463
                                                      properties)
 
1464
                            for client, properties
 
1465
                            in self.clients.items()))
 
1466
 
 
1467
    def test_is_enabled_run_exits_successfully(self):
2415
1468
        with self.assertRaises(SystemExit) as e:
2416
 
            command.IsEnabled().run(self.bus.one_client)
 
1469
            command.IsEnabled().run(self.one_client)
2417
1470
        if e.exception.code is not None:
2418
 
            self.assertEqual(0, e.exception.code)
 
1471
            self.assertEqual(e.exception.code, 0)
2419
1472
        else:
2420
1473
            self.assertIsNone(e.exception.code)
2421
1474
 
2422
 
    def test_IsEnabled_exits_with_failure(self):
2423
 
        self.bus.client_properties["Enabled"] = False
 
1475
    def test_is_enabled_run_exits_with_failure(self):
 
1476
        self.client.attributes["Enabled"] = dbus.Boolean(False)
2424
1477
        with self.assertRaises(SystemExit) as e:
2425
 
            command.IsEnabled().run(self.bus.one_client)
 
1478
            command.IsEnabled().run(self.one_client)
2426
1479
        if isinstance(e.exception.code, int):
2427
 
            self.assertNotEqual(0, e.exception.code)
 
1480
            self.assertNotEqual(e.exception.code, 0)
2428
1481
        else:
2429
1482
            self.assertIsNotNone(e.exception.code)
2430
1483
 
2431
 
    def test_Approve(self):
2432
 
        busname = "se.recompile.Mandos"
2433
 
        client_interface = "se.recompile.Mandos.Client"
2434
 
        command.Approve().run(self.bus.clients, self.bus)
2435
 
        for clientpath in self.bus.clients:
2436
 
            self.assertIn(("Approve", busname, clientpath,
2437
 
                           client_interface, (True,)), self.bus.calls)
2438
 
 
2439
 
    def test_Deny(self):
2440
 
        busname = "se.recompile.Mandos"
2441
 
        client_interface = "se.recompile.Mandos.Client"
2442
 
        command.Deny().run(self.bus.clients, self.bus)
2443
 
        for clientpath in self.bus.clients:
2444
 
            self.assertIn(("Approve", busname, clientpath,
2445
 
                           client_interface, (False,)),
2446
 
                          self.bus.calls)
2447
 
 
2448
 
    def test_Remove(self):
2449
 
        command.Remove().run(self.bus.clients, self.bus)
2450
 
        for clientpath in self.bus.clients:
2451
 
            self.assertIn(("RemoveClient", dbus_busname,
2452
 
                           dbus_server_path, dbus_server_interface,
2453
 
                           (clientpath,)), self.bus.calls)
 
1484
    def test_approve(self):
 
1485
        command.Approve().run(self.clients, self.bus)
 
1486
        for clientpath in self.clients:
 
1487
            client = self.bus.get_object(dbus_busname, clientpath)
 
1488
            self.assertIn(("Approve", (True, client_dbus_interface)),
 
1489
                          client.calls)
 
1490
 
 
1491
    def test_deny(self):
 
1492
        command.Deny().run(self.clients, self.bus)
 
1493
        for clientpath in self.clients:
 
1494
            client = self.bus.get_object(dbus_busname, clientpath)
 
1495
            self.assertIn(("Approve", (False, client_dbus_interface)),
 
1496
                          client.calls)
 
1497
 
 
1498
    def test_remove(self):
 
1499
        class MockMandos(object):
 
1500
            def __init__(self):
 
1501
                self.calls = []
 
1502
            def RemoveClient(self, dbus_path):
 
1503
                self.calls.append(("RemoveClient", (dbus_path,)))
 
1504
        mandos = MockMandos()
 
1505
        super(TestBaseCommands, self).setUp()
 
1506
        command.Remove().run(self.clients, self.bus, mandos)
 
1507
        self.assertEqual(len(mandos.calls), 2)
 
1508
        for clientpath in self.clients:
 
1509
            self.assertIn(("RemoveClient", (clientpath,)),
 
1510
                          mandos.calls)
2454
1511
 
2455
1512
    expected_json = {
2456
1513
        "foo": {
2504
1561
    }
2505
1562
 
2506
1563
    def test_DumpJSON_normal(self):
2507
 
        with self.capture_stdout_to_buffer() as buffer:
2508
 
            command.DumpJSON().run(self.bus.clients)
2509
 
        json_data = json.loads(buffer.getvalue())
2510
 
        self.assertDictEqual(self.expected_json, json_data)
2511
 
 
2512
 
    @staticmethod
2513
 
    @contextlib.contextmanager
2514
 
    def capture_stdout_to_buffer():
2515
 
        capture_buffer = io.StringIO()
2516
 
        old_stdout = sys.stdout
2517
 
        sys.stdout = capture_buffer
2518
 
        try:
2519
 
            yield capture_buffer
2520
 
        finally:
2521
 
            sys.stdout = old_stdout
 
1564
        output = command.DumpJSON().output(self.clients.values())
 
1565
        json_data = json.loads(output)
 
1566
        self.assertDictEqual(json_data, self.expected_json)
2522
1567
 
2523
1568
    def test_DumpJSON_one_client(self):
2524
 
        with self.capture_stdout_to_buffer() as buffer:
2525
 
            command.DumpJSON().run(self.bus.one_client)
2526
 
        json_data = json.loads(buffer.getvalue())
 
1569
        output = command.DumpJSON().output(self.one_client.values())
 
1570
        json_data = json.loads(output)
2527
1571
        expected_json = {"foo": self.expected_json["foo"]}
2528
 
        self.assertDictEqual(expected_json, json_data)
 
1572
        self.assertDictEqual(json_data, expected_json)
2529
1573
 
2530
1574
    def test_PrintTable_normal(self):
2531
 
        with self.capture_stdout_to_buffer() as buffer:
2532
 
            command.PrintTable().run(self.bus.clients)
 
1575
        output = command.PrintTable().output(self.clients.values())
2533
1576
        expected_output = "\n".join((
2534
1577
            "Name   Enabled Timeout  Last Successful Check",
2535
1578
            "foo    Yes     00:05:00 2019-02-03T00:00:00  ",
2536
1579
            "barbar Yes     00:05:00 2019-02-04T00:00:00  ",
2537
 
        )) + "\n"
2538
 
        self.assertEqual(expected_output, buffer.getvalue())
 
1580
        ))
 
1581
        self.assertEqual(output, expected_output)
2539
1582
 
2540
1583
    def test_PrintTable_verbose(self):
2541
 
        with self.capture_stdout_to_buffer() as buffer:
2542
 
            command.PrintTable(verbose=True).run(self.bus.clients)
 
1584
        output = command.PrintTable(verbose=True).output(
 
1585
            self.clients.values())
2543
1586
        columns = (
2544
1587
            (
2545
1588
                "Name   ",
2627
1670
            )
2628
1671
        )
2629
1672
        num_lines = max(len(rows) for rows in columns)
2630
 
        expected_output = ("\n".join("".join(rows[line]
2631
 
                                             for rows in columns)
2632
 
                                     for line in range(num_lines))
2633
 
                           + "\n")
2634
 
        self.assertEqual(expected_output, buffer.getvalue())
 
1673
        expected_output = "\n".join("".join(rows[line]
 
1674
                                            for rows in columns)
 
1675
                                    for line in range(num_lines))
 
1676
        self.assertEqual(output, expected_output)
2635
1677
 
2636
1678
    def test_PrintTable_one_client(self):
2637
 
        with self.capture_stdout_to_buffer() as buffer:
2638
 
            command.PrintTable().run(self.bus.one_client)
 
1679
        output = command.PrintTable().output(self.one_client.values())
2639
1680
        expected_output = "\n".join((
2640
1681
            "Name Enabled Timeout  Last Successful Check",
2641
1682
            "foo  Yes     00:05:00 2019-02-03T00:00:00  ",
2642
 
        )) + "\n"
2643
 
        self.assertEqual(expected_output, buffer.getvalue())
2644
 
 
2645
 
 
2646
 
class TestPropertySetterCmd(TestCommand):
2647
 
    """Abstract class for tests of command.PropertySetter classes"""
2648
 
 
 
1683
        ))
 
1684
        self.assertEqual(output, expected_output)
 
1685
 
 
1686
 
 
1687
class TestPropertyCmd(TestCommand):
 
1688
    """Abstract class for tests of command.Property classes"""
2649
1689
    def runTest(self):
2650
1690
        if not hasattr(self, "command"):
2651
 
            return              # Abstract TestCase class
2652
 
 
2653
 
        if hasattr(self, "values_to_set"):
2654
 
            cmd_args = [(value,) for value in self.values_to_set]
2655
 
            values_to_get = getattr(self, "values_to_get",
2656
 
                                    self.values_to_set)
2657
 
        else:
2658
 
            cmd_args = [() for x in range(len(self.values_to_get))]
2659
 
            values_to_get = self.values_to_get
2660
 
        for value_to_get, cmd_arg in zip(values_to_get, cmd_args):
2661
 
            for clientpath in self.bus.clients:
2662
 
                self.bus.clients[clientpath][self.propname] = (
2663
 
                    Unique())
2664
 
            self.command(*cmd_arg).run(self.bus.clients, self.bus)
2665
 
            for clientpath in self.bus.clients:
2666
 
                value = (self.bus.clients[clientpath]
2667
 
                         [self.propname])
2668
 
                self.assertNotIsInstance(value, Unique)
2669
 
                self.assertEqual(value_to_get, value)
2670
 
 
2671
 
 
2672
 
class TestEnableCmd(TestPropertySetterCmd):
 
1691
            return
 
1692
        values_to_get = getattr(self, "values_to_get",
 
1693
                                self.values_to_set)
 
1694
        for value_to_set, value_to_get in zip(self.values_to_set,
 
1695
                                              values_to_get):
 
1696
            for clientpath in self.clients:
 
1697
                client = self.bus.get_object(dbus_busname, clientpath)
 
1698
                old_value = client.attributes[self.propname]
 
1699
                self.assertNotIsInstance(old_value, self.Unique)
 
1700
                client.attributes[self.propname] = self.Unique()
 
1701
            self.run_command(value_to_set, self.clients)
 
1702
            for clientpath in self.clients:
 
1703
                client = self.bus.get_object(dbus_busname, clientpath)
 
1704
                value = client.attributes[self.propname]
 
1705
                self.assertNotIsInstance(value, self.Unique)
 
1706
                self.assertEqual(value, value_to_get)
 
1707
 
 
1708
    class Unique(object):
 
1709
        """Class for objects which exist only to be unique objects,
 
1710
since unittest.mock.sentinel only exists in Python 3.3"""
 
1711
 
 
1712
    def run_command(self, value, clients):
 
1713
        self.command().run(clients, self.bus)
 
1714
 
 
1715
 
 
1716
class TestEnableCmd(TestPropertyCmd):
2673
1717
    command = command.Enable
2674
1718
    propname = "Enabled"
2675
 
    values_to_get = [True]
2676
 
 
2677
 
 
2678
 
class TestDisableCmd(TestPropertySetterCmd):
 
1719
    values_to_set = [dbus.Boolean(True)]
 
1720
 
 
1721
 
 
1722
class TestDisableCmd(TestPropertyCmd):
2679
1723
    command = command.Disable
2680
1724
    propname = "Enabled"
2681
 
    values_to_get = [False]
2682
 
 
2683
 
 
2684
 
class TestBumpTimeoutCmd(TestPropertySetterCmd):
 
1725
    values_to_set = [dbus.Boolean(False)]
 
1726
 
 
1727
 
 
1728
class TestBumpTimeoutCmd(TestPropertyCmd):
2685
1729
    command = command.BumpTimeout
2686
1730
    propname = "LastCheckedOK"
2687
 
    values_to_get = [""]
2688
 
 
2689
 
 
2690
 
class TestStartCheckerCmd(TestPropertySetterCmd):
 
1731
    values_to_set = [""]
 
1732
 
 
1733
 
 
1734
class TestStartCheckerCmd(TestPropertyCmd):
2691
1735
    command = command.StartChecker
2692
1736
    propname = "CheckerRunning"
2693
 
    values_to_get = [True]
2694
 
 
2695
 
 
2696
 
class TestStopCheckerCmd(TestPropertySetterCmd):
 
1737
    values_to_set = [dbus.Boolean(True)]
 
1738
 
 
1739
 
 
1740
class TestStopCheckerCmd(TestPropertyCmd):
2697
1741
    command = command.StopChecker
2698
1742
    propname = "CheckerRunning"
2699
 
    values_to_get = [False]
2700
 
 
2701
 
 
2702
 
class TestApproveByDefaultCmd(TestPropertySetterCmd):
 
1743
    values_to_set = [dbus.Boolean(False)]
 
1744
 
 
1745
 
 
1746
class TestApproveByDefaultCmd(TestPropertyCmd):
2703
1747
    command = command.ApproveByDefault
2704
1748
    propname = "ApprovedByDefault"
2705
 
    values_to_get = [True]
2706
 
 
2707
 
 
2708
 
class TestDenyByDefaultCmd(TestPropertySetterCmd):
 
1749
    values_to_set = [dbus.Boolean(True)]
 
1750
 
 
1751
 
 
1752
class TestDenyByDefaultCmd(TestPropertyCmd):
2709
1753
    command = command.DenyByDefault
2710
1754
    propname = "ApprovedByDefault"
2711
 
    values_to_get = [False]
2712
 
 
2713
 
 
2714
 
class TestSetCheckerCmd(TestPropertySetterCmd):
 
1755
    values_to_set = [dbus.Boolean(False)]
 
1756
 
 
1757
 
 
1758
class TestPropertyValueCmd(TestPropertyCmd):
 
1759
    """Abstract class for tests of PropertyValueCmd classes"""
 
1760
 
 
1761
    def runTest(self):
 
1762
        if type(self) is TestPropertyValueCmd:
 
1763
            return
 
1764
        return super(TestPropertyValueCmd, self).runTest()
 
1765
 
 
1766
    def run_command(self, value, clients):
 
1767
        self.command(value).run(clients, self.bus)
 
1768
 
 
1769
 
 
1770
class TestSetCheckerCmd(TestPropertyValueCmd):
2715
1771
    command = command.SetChecker
2716
1772
    propname = "Checker"
2717
1773
    values_to_set = ["", ":", "fping -q -- %s"]
2718
1774
 
2719
1775
 
2720
 
class TestSetHostCmd(TestPropertySetterCmd):
 
1776
class TestSetHostCmd(TestPropertyValueCmd):
2721
1777
    command = command.SetHost
2722
1778
    propname = "Host"
2723
 
    values_to_set = ["192.0.2.3", "client.example.org"]
2724
 
 
2725
 
 
2726
 
class TestSetSecretCmd(TestPropertySetterCmd):
 
1779
    values_to_set = ["192.0.2.3", "foo.example.org"]
 
1780
 
 
1781
 
 
1782
class TestSetSecretCmd(TestPropertyValueCmd):
2727
1783
    command = command.SetSecret
2728
1784
    propname = "Secret"
2729
1785
    values_to_set = [io.BytesIO(b""),
2730
1786
                     io.BytesIO(b"secret\0xyzzy\nbar")]
2731
 
    values_to_get = [f.getvalue() for f in values_to_set]
2732
 
 
2733
 
 
2734
 
class TestSetTimeoutCmd(TestPropertySetterCmd):
 
1787
    values_to_get = [b"", b"secret\0xyzzy\nbar"]
 
1788
 
 
1789
 
 
1790
class TestSetTimeoutCmd(TestPropertyValueCmd):
2735
1791
    command = command.SetTimeout
2736
1792
    propname = "Timeout"
2737
1793
    values_to_set = [datetime.timedelta(),
2739
1795
                     datetime.timedelta(seconds=1),
2740
1796
                     datetime.timedelta(weeks=1),
2741
1797
                     datetime.timedelta(weeks=52)]
2742
 
    values_to_get = [dt.total_seconds()*1000 for dt in values_to_set]
2743
 
 
2744
 
 
2745
 
class TestSetExtendedTimeoutCmd(TestPropertySetterCmd):
 
1798
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1799
 
 
1800
 
 
1801
class TestSetExtendedTimeoutCmd(TestPropertyValueCmd):
2746
1802
    command = command.SetExtendedTimeout
2747
1803
    propname = "ExtendedTimeout"
2748
1804
    values_to_set = [datetime.timedelta(),
2750
1806
                     datetime.timedelta(seconds=1),
2751
1807
                     datetime.timedelta(weeks=1),
2752
1808
                     datetime.timedelta(weeks=52)]
2753
 
    values_to_get = [dt.total_seconds()*1000 for dt in values_to_set]
2754
 
 
2755
 
 
2756
 
class TestSetIntervalCmd(TestPropertySetterCmd):
 
1809
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1810
 
 
1811
 
 
1812
class TestSetIntervalCmd(TestPropertyValueCmd):
2757
1813
    command = command.SetInterval
2758
1814
    propname = "Interval"
2759
1815
    values_to_set = [datetime.timedelta(),
2761
1817
                     datetime.timedelta(seconds=1),
2762
1818
                     datetime.timedelta(weeks=1),
2763
1819
                     datetime.timedelta(weeks=52)]
2764
 
    values_to_get = [dt.total_seconds()*1000 for dt in values_to_set]
2765
 
 
2766
 
 
2767
 
class TestSetApprovalDelayCmd(TestPropertySetterCmd):
 
1820
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1821
 
 
1822
 
 
1823
class TestSetApprovalDelayCmd(TestPropertyValueCmd):
2768
1824
    command = command.SetApprovalDelay
2769
1825
    propname = "ApprovalDelay"
2770
1826
    values_to_set = [datetime.timedelta(),
2772
1828
                     datetime.timedelta(seconds=1),
2773
1829
                     datetime.timedelta(weeks=1),
2774
1830
                     datetime.timedelta(weeks=52)]
2775
 
    values_to_get = [dt.total_seconds()*1000 for dt in values_to_set]
2776
 
 
2777
 
 
2778
 
class TestSetApprovalDurationCmd(TestPropertySetterCmd):
 
1831
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
 
1832
 
 
1833
 
 
1834
class TestSetApprovalDurationCmd(TestPropertyValueCmd):
2779
1835
    command = command.SetApprovalDuration
2780
1836
    propname = "ApprovalDuration"
2781
1837
    values_to_set = [datetime.timedelta(),
2783
1839
                     datetime.timedelta(seconds=1),
2784
1840
                     datetime.timedelta(weeks=1),
2785
1841
                     datetime.timedelta(weeks=52)]
2786
 
    values_to_get = [dt.total_seconds()*1000 for dt in values_to_set]
 
1842
    values_to_get = [0, 300000, 1000, 604800000, 31449600000]
2787
1843
 
2788
1844
 
2789
1845