/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-02-28 19:22:08 UTC
  • Revision ID: teddy@recompile.se-20190228192208-1bkaewjx1vzar0sl
mandos-ctl: Use logging module instead of print() for errors

* mandos-ctl (log): New logging object.
  (main): Use log instead of print("<message>", file=sys.stderr).

Show diffs side-by-side

added added

removed removed

Lines of Context:
mandos-ctl
41
41
import collections
42
42
import json
43
43
import unittest
 
44
import logging
44
45
 
45
46
import dbus
46
47
 
 
48
log = logging.getLogger(sys.argv[0])
 
49
logging.basicConfig(level="INFO", # Show info level messages
 
50
                    format="%(message)s") # Show basic log messages
 
51
 
47
52
if sys.version_info.major == 2:
48
53
    str = unicode
49
54
 
350
355
        bus = dbus.SystemBus()
351
356
        mandos_dbus_objc = bus.get_object(busname, server_path)
352
357
    except dbus.exceptions.DBusException:
353
 
        print("Could not connect to Mandos server", file=sys.stderr)
 
358
        log.critical("Could not connect to Mandos server")
354
359
        sys.exit(1)
355
360
 
356
361
    mandos_serv = dbus.Interface(mandos_dbus_objc,
375
380
            os.dup2(stderrcopy, sys.stderr.fileno())
376
381
            os.close(stderrcopy)
377
382
    except dbus.exceptions.DBusException as e:
378
 
        print("Access denied: "
379
 
              "Accessing mandos server through D-Bus: {}".format(e),
380
 
              file=sys.stderr)
 
383
        log.critical("Failed to access Mandos server through D-Bus:"
 
384
                     "\n%s", e)
381
385
        sys.exit(1)
382
386
 
383
387
    # Compile dict of (clients: properties) to process
394
398
                    clients[client_objc] = client
395
399
                    break
396
400
            else:
397
 
                print("Client not found on server: {!r}"
398
 
                      .format(name), file=sys.stderr)
 
401
                log.critical("Client not found on server: %r", name)
399
402
                sys.exit(1)
400
403
 
401
404
    if not has_actions(options) and clients: