Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update view_report() #336

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update view_report() #336

wants to merge 1 commit into from

Conversation

kdubdev
Copy link

@kdubdev kdubdev commented Apr 30, 2019

Modified view_report() with following changes:

  • added option 'newest' as alias to --report and --report "" to allow $ maldet --report newest user@domain.com
  • properly email most recent report when using $ maldet --report newest user@domain.com or $ maldet --report "" user@domain.com
  • change from unnecessarily using editor to view reports, instead simply print to terminal
  • improved logging

This was to address long standing issues discussed here: https://serverfault.com/questions/805158/how-to-get-an-email-report-of-whatever-the-most-recent-maldet-scan-is

Modified view_report() with following changes:
* added arg 'newest' as alias to --report and --report "" to allow $ maldet --report newest user@domain.com
* properly email most recent report when using $ maldet --report newest user@domain.com or $ maldet --report "" user@domain.com
* change from using editor to view reports to simply print to terminal
This was to address long standing issues discussed here: https://serverfault.com/questions/805158/how-to-get-an-email-report-of-whatever-the-most-recent-maldet-scan-is
@rfxn rfxn self-requested a review May 5, 2019 04:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants