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.

Sign up for GitHub

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

Jump to bottom

sympa_msg.pl: Rejection reports are suppressed #820

Closed
ikedas opened this issue Dec 10, 2019 · 0 comments · Fixed by #821
Closed

sympa_msg.pl: Rejection reports are suppressed #820

ikedas opened this issue Dec 10, 2019 · 0 comments · Fixed by #821
Labels
bug ready A PR is waiting to be merged. Close to be solved
Milestone
6.2.50

Comments

@ikedas
Copy link
Member

ikedas commented Dec 10, 2019

Version

I confirmed this bug with 6.2.40.

According to code review, perhaps 6.2.13 to recent (6.2.49b.2) are alike.

Installation method

RPM, but any methods may apply.

Expected behavior

If a "reject" action in a scenario does not have "quiet" modifier, and the incoming message does not have spam flag, rejection notice will be sent back.

Actual behavior

After a while, rejection notice will no longer be sent back.

Additional information

I found that, once any actions (except "reject") with "quiet" modifier has been applied, rejection notice will be suppressed.

I'm preparing a PR to fix this behavior.
@ikedas ikedas added the bug label Dec 10, 2019
@ikedas ikedas added this to the 6.2.50 milestone Dec 10, 2019
ikedas added a commit to ikedas/sympa that referenced this issue Dec 10, 2019
@ikedas
Fixes sympa-community#820: If a "reject" action in a scenario does no…
cdc72cb 
…t have "quiet" modifier, and the incoming message does not have spam flag, rejection notice will be sent back.
@ikedas ikedas mentioned this issue Dec 10, 2019
Merged
@ikedas ikedas added the ready A PR is waiting to be merged. Close to be solved label Dec 10, 2019
ikedas added a commit that referenced this issue Dec 13, 2019
@ikedas
Merge pull request #821 from ikedas/messages_quietly_rejected by ikedas
3623a25 
Fixes issue #820: sympa_msg.pl: Rejection reports were suppressed
@ikedas ikedas mentioned this issue Jan 27, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug ready A PR is waiting to be merged. Close to be solved
Projects
None yet
Milestone
6.2.50
Development

Successfully merging a pull request may close this issue.

Fixes #820 ikedas/sympa
1 participant
@ikedas