Skip to content

Conversation

@printminion-co
Copy link

@printminion-co printminion-co commented Jan 29, 2025

Test

  • in local dev or via ./<nc-dev-and-tools>/container/check_release.sh pr-93
  • enable admin_audit ./occ app:enable admin_audit
  • execute any occ command e.g. ./occ app:list. Observe Console output
[hidrivenext_admin_audit][admin_audit][1] {"reqId":"3YOgwClfLQHPZPk7ER1n","level":1,"time":"2025-01-29T10:53:25+00:00","remoteAddr":"","user":"--","app":"admin_audit","method":"","url":"--","message":"Console command executed: app:list","userAgent":"--","version":"30.0.0.14","data":{"app":"admin_audit"}}

TODO

Copy link

@bromiesTM bromiesTM left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should enable the app via our apps-enable script
Please add it to IONOS-Productivity/nc-config#46

Edit: This is being done separately

@printminion-co printminion-co force-pushed the mk/dev/add_audit_log_config branch from e3e0911 to 4e7f5fb Compare January 29, 2025 16:21
* feature(admin_audit): add admin_audit config values
* feat(apps-enable-core): ensure required core apps are enabled

Signed-off-by: Misha M.-Kupriyanov <[email protected]>
@printminion-co printminion-co force-pushed the mk/dev/add_audit_log_config branch from 4e7f5fb to 21b2d74 Compare January 30, 2025 09:44
@printminion-co printminion-co merged commit 4ac8d00 into ionos-dev Jan 30, 2025
5 of 6 checks passed
@printminion-co printminion-co deleted the mk/dev/add_audit_log_config branch January 30, 2025 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants