URL:
https://github.com/SSSD/sssd/pull/5712
Title: #5712: Health and Support Analyzer - Add request log parsing utility
pbrezina commented:
"""
This looks better, thanks. I would certainly like to get it reviewed by python developers.
At least, the two Reader classes should use inheritance.
Perhaps, we can avoid using @PYTHON_BIN_PATH@ and configure time option and just sed the
shebang in spec file after install? I'm not sure if this is a good idea, please reach
to python developers for advice. But it would make testing easier.
I think it would be also good to print uid of the user who started the request (client
uid) in the `list` command. It is not yet available in the logs but can be added from
`cli_creds_get_uid(cli_ctx->creds)`
"""
See the full comment at
https://github.com/SSSD/sssd/pull/5712#issuecomment-932135349