Skip to content

Cli log

Svein Tore Koksrud Seljebotn requested to merge cli-log into dev

Description

Closes issues: Related issues:

Adds a new logging table to database which logs usage of the CLI commands which usage should be logged for audit purposes.

  • Moves ella-cli to new bin/ dir. Added to PATH in Docker image.
  • Adds auto-completion for ella-cli to bash profile in Docker image
  • Adds table clilog
  • Logs most CLI commands to said table
  • Some commands has a optional reason flag for stating the reason for issuing the command

Notes to reviewer

  • We might want to capture all stdout usage, but I believe it's a bit of work and can be fragile (?)
  • Removed add single user since the function was broken, and it shouldn't really be used.

Type of change

  • New feature
  • Breaking change
  • Bug fix
  • Improvement

Testing

  • Tests have been added that prove my fix is effective or that my feature works
  • Related tests have been modified/removed

Database

  • Includes changes to database schema
  • Includes necessary database migrations

Configuration

  • Includes changes to configuration
  • Includes configuration migration instructions in documentation

General

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My branch has recently been updated with new changes in dev before this merge request
  • I have made corresponding changes to the documentation
Edited by Svein Tore Koksrud Seljebotn

Merge request reports

Loading