Opened 7 years ago

Closed 5 years ago

#2620 closed defect (wontfix)

b10-cmdctl-usermgr should check permissions on start

Reported by: shane Owned by:
Priority: medium Milestone: Remaining BIND10 tickets
Component: Unclassified Version: bind10-old
Keywords: Cc:
CVSS Scoring: Parent Tickets:
Sensitive: no Defect Severity: Low
Sub-Project: Core Feature Depending on Ticket:
Estimated Difficulty: 3 Add Hours to Ticket: 0
Total Hours: 0 Internal?: no

Description

I tried to add a user:

$ ./b10-cmdctl-usermgr 
Desired Login Name:shane
Choose a password:
Re-enter password:
Traceback (most recent call last):
  File "./b10-cmdctl-usermgr", line 124, in <module>
    main()
  File "./b10-cmdctl-usermgr", line 114, in main
    save_userinfo(name, pw, salt, filename)
  File "./b10-cmdctl-usermgr", line 59, in save_userinfo
    csvfile = open(filename, 'a')
IOError: [Errno 13] Permission denied: 'cmdctl-accounts.csv'

There should be 2 changes:

  1. The program should check permissions when it starts, so I don't go through the steps of adding a user when they won't work.
  1. If for some other reason updating user information fails (permissions can change after the program starts, or NFS mounts can disappear, or whatever), the program should catch the problem and give a nicer error rather than dumping a stack trace.

Subtickets

Change History (3)

comment:1 Changed 7 years ago by shane

  • Milestone changed from New Tasks to Next-Sprint-Proposed

comment:2 Changed 6 years ago by tomek

  • Milestone set to Remaining BIND10 tickets

comment:3 Changed 5 years ago by tomek

  • Resolution set to wontfix
  • Status changed from new to closed
  • Version set to old-bind10

This issue is related to bind10 code that is no longer part of Kea.

If you are interested in BIND10/Bundy framework or its DNS components,
please check http://bundy-dns.de.

Closing ticket.

Note: See TracTickets for help on using tickets.