Analysis time button bug fix

[egbicker]

Fixes to allow the time selection button to properly filter syslog events.

Closes #832

[egbicker]

@jw3 I think there might be an issue when parsing the when variable in the Rust layer. I added my best guess at a print statement in my most recent commit to see the value of e.rs.event.when in crates/pyo3/src/analysis.rs temporal_filter function and got None. Looking at sudo systemctl status fapolicyd.service and sudo journalctl it looks like the time format is not rfc3339.

[jw3]

Whats this say grep FileFormat /etc/rsyslog.conf

[egbicker]

module(load="builtin:omfile" Template="RSYSLOG_TraditionalFileFormat")

[jw3]

#868 describes what you are seeing.

Quick fix is to update the syslog format. The playbook does that, I am guessing you didnt set this vm up from there.

- name: set syslog format
  replace:
    path: /etc/rsyslog.conf
    regexp: "RSYSLOG_TraditionalFileFormat"
    replace: "RSYSLOG_FileFormat"

[jw3]

Btw, this is a good find 👍 .

That format change was made way back in development of the analysis view, I had forgot about it since then, but it definitely needs to be more robust.

[egbicker]

@jw3 This seems to be working after I cleared out the entries with the old format. Do you think its worth adding that grep as a check before enabling the time selection or is it better to open an issue with the parsing?

[jw3]

Maybe the analyzer should warn the user if entries come back without a when value.

You could detect that from the returned results and display a dialog that mentions the config items to check.

[dorschs57]

One minor suggested change. Otherwise the code looks good.

[dorschs57]

Code looks good 👍