[issue2551246] Implement schema permissions for roundup-admin (possibly password too)

John Rouillard <[email protected]> Wed, 30 Nov 2022 07:05:28 +0000
Newsgroups gmane.comp.bug-tracking.roundup.devel
Message-ID <[email protected]>
New submission from John Rouillard:

roundup-admin has a -u option that supposedly sets the user. However it does not
appear to actually apply the permissions schema that is defined.

This isn't a huge problem as it would be for the html interface. In order to use
roundup-admin you have to have access to the home directory at the OS level. If you
have that you have the password to the database in config.ini. So all this bug does
is make using something like:

   sudo -u roundup roundup-admin -u report -i demo

not work as expected. If the report user changes data, the history does log that it
was the report user who did the change.

For example:

  roundup-admin -u anonymous -i demo table user id,username,password

should return the password entry for the anonymous user only.
It returns all passwords.

  roundup-admin -u anonymous display user1

should not display password etc. It returns all the info (password, addresses etc.)

Some commands kind of work. For example:

  roundup_admin -u foo -i demo history user1
  Error: no such class "user"

where the user foo doesn't exist is a bit misleading, but doesn't disclose the
history. Users demo and anonymous display the info (as in the web interface), so
the command is valid.

Also there is no need for a password (username:password) that is compared against
the database. Not sure that a password is of much use since user validation would
be done at the sudo level but there might be a valid use case.

I updated the docs with a warning and added a note to the inline help in admin.py.

----------
components: Command-line interface
messages: 7689
nosy: rouilj
severity: minor
status: new
title: Implement schema permissions for roundup-admin (possibly password too)
type: behavior

_________________________________________________
Roundup tracker <[email protected]>
<https://issues.roundup-tracker.org/issue2551246>
_________________________________________________