Re: [PATCH] [white|black]list for receiver addresses

Stipe Tolj <[email protected]>
Newsgroups gmane.comp.mobile.kannel.devel
Organization Kannel Foundation (KSF)
Message-ID <[email protected]>
Am 31.10.2014 11:30, schrieb Alexander Malysh:
> Hi,
>
> +1 for the feature but not for the patch itself.
>
> I would propose to make it clear in the code and userguide which list ist for which field. Please rename black-list… to black-list-sender etc.
> For the migration phase you can accept in the config both, black-list-sender and black-list and warn about deprecated config values.
>
> We have to keep code clean…

fine with me, I'll re-factor the config directive names and go for commit.

Stipe

-- 
-------------------------------------------------------------------
Kölner Landstrasse 419
40589 Düsseldorf, NRW, Germany

Kannel Foundation                 tolj.org system architecture
http://www.kannel.org/            http://www.tolj.org/

mailto:stolj_{at}_kannel.org      mailto:st_{at}_tolj.org
-------------------------------------------------------------------
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.