Skip to content
This repository has been archived by the owner on May 10, 2023. It is now read-only.

Let extensions define their own statistics #3

Open
tonioo opened this issue May 5, 2015 · 5 comments
Open

Let extensions define their own statistics #3

tonioo opened this issue May 5, 2015 · 5 comments
Assignees

Comments

@tonioo
Copy link
Member

tonioo commented May 5, 2015

From @tonioo on December 4, 2013 15:45

Originally created by Antoine Nguyen on 2012-10-21T10:12:17Z

It could be useful to let external users (applications, extensions) define new statistics (graphics).

Copied from original issue: modoboa/modoboa#331

@xanderificnl
Copy link

Are there concrete plans to implement this?

I've just set-up SQLGrey to lower the amount of spam I've been receiving and I plan on writing a frontend for Modoboa and I would like to add greylisting to stats.

By default, greylistings show up as rejected - which technically is accurate.

I've fiddled a bit with the code on a test box - and modified the stats directly but it's not pretty at all.

selection_046

@tonioo
Copy link
Member Author

tonioo commented Sep 18, 2015

There is not real plan to implement this feature right now.

Anyway, since greylisting is something people use, you can propose a patch to add this metric into the default graph. We could also enable/disable it with a setting. What do you think ?

@xanderificnl
Copy link

I'm not familiar with other tools (I've only used SQLGrey) but I can imagine that other tools use a different message to reject; so it might be prudent to introduce a 'greylisting regex' as a setting as well, for those who've either customized the message (which is an option in SQLGrey) and those who use different software.

I'll need to lookup the proper documentation on creating an option. I'll try to submit a PR soon.

Though, in the long run - a feature such as described here would be the way to go.

@mundschenk-at
Copy link
Contributor

@web-refinery Any chance that we might get that PR? I'd love to see the greylisted mails separately.

@mundschenk-at
Copy link
Contributor

@tonioo @web-refinery I've added PR #24 to introduce greylisting support. It would need support for different greylisting implementations (at least postgrey) to be widely useful - currently it only support rspamd in conjunction with rmilter.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants