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

Email Alert message with attach report option does not allow use of $i for full report text #2072

Closed
syspect-sec opened this issue Aug 29, 2023 · 1 comment
Labels
bug Something isn't working

Comments

@syspect-sec
Copy link

Expected behavior

According to the manual, the $i dynamic report information variable should work with both
include and attach report reporting options according to the docs here

Actual behavior

Although the $i variable is honoured with the include report option it is not honoured for the attach report option.
In the attach report case, the emailed report does not include the full report text into the email body.
This impacts all installations tested so far:

  • Ubuntu source code install
  • Ubuntu Docker containers
  • Kali Linux native GVM installation
  • CentOS source code install

Steps to reproduce

  1. Configure Greenbone with SMTP configuration (can use this tutorial here)
  2. Create an alert and select Email method with the attach report option
  3. Select any report attachment option
  4. Add the $i variable to the report text configuration input textbox
  5. Trigger a test of the alert

Environment

Operating system:

This impacts all installations tested so far:

  • Ubuntu source code install
  • Ubuntu Docker containers
  • Kali Linux native GVM installation
  • CentOS source code install

Logfiles

No relevant output in the logs.

@syspect-sec syspect-sec added the bug Something isn't working label Aug 29, 2023
@mattmundell
Copy link
Contributor

Thanks for the clear description.

The $i variable is intentionally only supported for the include report option.

I guess the doc you linked has been updated since you reported, it currently says:

    $i: the report text or list of SecInfo objects (only if including the report/list).

"only if including" refers to the include report option.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants