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

Spamassassin spam learning integration #408

Open
wants to merge 5 commits into
base: dev
Choose a base branch
from

Conversation

pboguslawski
Copy link
Contributor

Proposed change

This mod adds new sysconfig parameter Ticket::EventModulePost###9996-LearnSpam that when enabled allows to mark ticket (using ticket text dynamic field PendingSpamLearningOperation which must exist in system, best as internal dynamic field) for learning as spam or as ham.

SpamQueues value in Ticket::EventModulePost###9996-LearnSpam must contain names of all spam queues (::: separated).

TrashQueues value in Ticket::EventModulePost###9996-LearnSpam must contain names of all trash queues (::: separated).

Ticked is marked for learning as spam (PendingSpamLearningOperation dynamic field set to "spam") when is moved from non-spam queue to spam queue.

Ticked is marked for learning as ham (PendingSpamLearningOperation dynamic field set to "ham") when is moved from spam or trash queue to non-spam, non-trash queue.

When option NewStateAfterMarkingSpam is defined, ticket state is changed to specified state after ticket is marked for learning as spam.

This mod introduces otrs.Console.pl Maint::Ticket::SpamAssassinLearn command (requires https://metacpan.org/pod/Mail::SpamAssassin::Client) to allow sending marked tickets (all customer e-mail messages from ticket) for learning in SpamAssassin service (PendingSpamLearningOperation is deleted from ticket after successfull learning). This task may be executed periodically by application daemon when parameter Daemon::SchedulerCronTaskManager::Task###SpamAssassinLearn is enabled and configured in SysConfig.

Type of change

  • '1 - 🚀 feature' - New feature (which adds functionality to an existing integration)

Additional information

Author-Change-Id: IB#1110581

Checklist

  • The code change is tested and works locally.(❗)
  • There is no commented out code in this PR.(❕)
  • You improved or added new unit tests.(❕)
  • Local ZnunyCodePolicy passed.(❕)
  • Local UnitTests / Selenium passed.(❕)
  • GitHub workflow CI (UnitTests / Selenium) passed.(❗)

This mod adds new sysconfig parameter Ticket::EventModulePost###9996-LearnSpam that
when enabled allows to mark ticket (using ticket text dynamic field
PendingSpamLearningOperation which must exist in system, best as internal dynamic
field) for learning as spam or as ham.

SpamQueues value in Ticket::EventModulePost###9996-LearnSpam must contain names
of all spam queues (::: separated).

TrashQueues value in Ticket::EventModulePost###9996-LearnSpam must contain names
of all trash queues (::: separated).

Ticked is marked for learning as spam (PendingSpamLearningOperation dynamic field
set to "spam") when is moved from non-spam queue to spam queue.

Ticked is marked for learning as ham (PendingSpamLearningOperation dynamic field
set to "ham") when is moved from spam or trash queue to non-spam, non-trash queue.

Author-Change-Id: IB#1110581
…arning in SpamAssassin)

This mod introduces otrs.Console.pl Maint::Ticket::SpamAssassinLearn command (requires
https://metacpan.org/pod/Mail::SpamAssassin::Client) to allow sending marked tickets
(all customer e-mail messages from ticket) for learning in SpamAssassin service
(PendingSpamLearningOperation is deleted from ticket after successfull learning).
This task may be executed periodically by application daemon when parameter
Daemon::SchedulerCronTaskManager::Task###SpamAssassinLearn is enabled and
configured in SysConfig.

Related: b99c723
Author-Change-Id: IB#1110581
NewStateAfterMarkingSpam option added to allow changing
ticket state after makring it for learning spam. It
allows for autoclosing tickets on move to spam queue.

Related: b99c723
Author-Change-Id: IB#1110581
Author-Change-Id: IB#1110581
Author-Change-Id: IB#1110581
@dennykorsukewitz dennykorsukewitz added the 1 - 🚀 feature New feature or request label Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - 🚀 feature New feature or request
Development

Successfully merging this pull request may close these issues.

2 participants