Ticket #307 (testing defect: fixed)

Opened 8 years ago

Last modified 5 years ago

Labeled spam should not be redelivered when marked as ham

Reported by: dmorton Owned by: mortonda@…
Priority: normal Milestone: 1.0.3
Component: PHP scripts Version: 1.0.1
Severity: normal Keywords: labeled rescue
Cc:

Description (last modified by mortonda@…) (diff)

When your settings are set to label spam, it delivers it; when you mark it

as ham, it gets delivered again.

Change History

Changed 8 years ago by anonymous

  • keywords labeled rescue added
  • milestone set to 1.0.2

Changed 8 years ago by rjl

Just recording David's suggestion for dealing with this issue:

"Perhaps we should add another "type", called "L" for labeled spam.... which the spam quarantine page would look for both S and L, but behave differently on rescue..."

This seems to me to be the best way to handle this issue properly, yes.

Changed 8 years ago by dmorton

  • description modified (diff)
  • milestone changed from 1.0.2 to 1.0.3

I think 'L' is being used for blacklisting. I also wonder if this ticket

would be obsoleted when we open up the scoring to have the better kill levels, aka 4th level patch.

Changed 8 years ago by anonymous

Another way to fix this which is impossible right now but possible with

per-address stats... Look at the matching policy and take action according to spam_lovers and/or spam_kill_level. But there's no way currently to track a maia_user id back to a linked user and policy.

Changed 8 years ago by anonymous

  • description modified (diff)

This was supposed ot be in the last comment, not the description:

In any case, this would have to touch a lot of code.... Let's bump this to 1.0.3 or 1.1

Changed 5 years ago by mortonda@…

  • description modified (diff)

I've tried coding a new type, (P), and it makes for some ugly code; It would probably be better to add a flag field to mmr to designated what happened to the message.

Changed 5 years ago by mortonda@…

  • owner changed from rjl to mortonda@…
  • status changed from new to accepted

[1391] works for 1.0.x. I still need to implement for 1.1.x, but I'm not sure if this is how I'd like to do it.

Changed 5 years ago by mortonda@…

  • status changed from accepted to testing
  • resolution set to fixed
Note: See TracTickets for help on using tickets.