Subject prefix

[LaVibeX]

Description

• Email Subject Prefix is Hardcoded rightnow .subject("[Dependency-Track] " + notification.getTitle()),
• This is a solution to allow the User to modify the Subject Prefix to its will .subject(smtpPrefix + " " + notification.getTitle() )

This modification would be available in Administration>Configuration>Email
and "[Dependency-Track]" would be the first default value.

If the User change the Subject Prefix, it will need to press Update button in order to save the changes.
Subject Prefix is not a required field so leaving the input box in blank is also an option.

This new feature helps to distinguish between several instances of Dependency-Track, e.g. prod and qual systems.

Default value: [Dependency-Track]

Modified Value: [Dependency-Track TEST ®]

Modified Value: " "

This PR depends on its corresponding PR in the frontend repository
DependencyTrack/frontend#720

Addressed Issue

This fixes #1823

Additional Details

Check if smtpPrefix is null in order to clean the value to " "
smtpPrefix = smtpPrefix == null ? " " : smtpPrefix;

This PR is supervised by @lukas-braune.

Checklist

• I have read and understand the contributing guidelines
• This PR fixes a defect, and I have provided tests to verify that the fix is effective
• This PR implements an enhancement, and I have provided tests to verify that it works as intended
• This PR introduces changes to the database model, and I have added corresponding update logic
• This PR introduces new or alters existing behavior, and I have updated the documentation accordingly

Signed-off-by: Andres Tito andres.tito@rohde-schwarz.com

[nscuro]

Thanks @LaVibeX!

[nscuro]

Looks like this broke SendMailPublisherTest. Do you mind having a look @LaVibeX?

[LaVibeX]

Locally, I ran the tests after my commit and it finish without any errors.
Here in the CI, I notice that the error is related to ProjectResourceTest due to a Condition TimeOut.

Could it be possible to run the test again @nscuro ?

Looks like this broke SendMailPublisherTest. Do you mind having a look @LaVibeX?

[nscuro]

Triggered another test run.

[girtsn]

it says that it implements #1823
from the ticket "add the notification subject configuration to the notification template and use same params as there exists for content"
is it possible to use variables now as in body of the notification (e.g. severity / others)? the description of this ticket seems to be about adding a static prefix only?

[nscuro]

You are right @girtsn, I reopened the issue. Thanks for pointing that out, much appreciated!