Browse Source

feat(scim): warn when SCIM runs without org event logging

log_event is a no-op unless ORG_EVENTS_ENABLED is set, so SCIM provisioning
changes would leave no audit trail on a default deployment. Print a startup
warning when SCIM_ENABLED is set but ORG_EVENTS_ENABLED is not.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
pull/7443/head
David Croft 3 days ago
parent
commit
2ac68220b0
  1. 5
      src/config.rs

5
src/config.rs

@ -1008,6 +1008,11 @@ fn validate_config(cfg: &ConfigItems, on_update: bool) -> Result<(), Error> {
println!("[WARNING] To enable the admin page without a token, use `DISABLE_ADMIN_TOKEN`."); println!("[WARNING] To enable the admin page without a token, use `DISABLE_ADMIN_TOKEN`.");
} }
if cfg.scim_enabled && !cfg.org_events_enabled {
println!("[WARNING] `SCIM_ENABLED` is set, but `ORG_EVENTS_ENABLED` is not.");
println!("[WARNING] SCIM provisioning changes will not appear in the organization event log.");
}
if cfg.push_enabled && (cfg.push_installation_id == String::new() || cfg.push_installation_key == String::new()) { if cfg.push_enabled && (cfg.push_installation_id == String::new() || cfg.push_installation_key == String::new()) {
err!( err!(
"Misconfigured Push Notification service\n\ "Misconfigured Push Notification service\n\

Loading…
Cancel
Save