[pmg-devel] [PATCH pmg-gui v2 1/1] MailProxyOptions: add greylist enhancements

[Dominik Csapak]

comment inline

On 4/20/20 1:22 PM, Stoiko Ivanov wrote:
> This patch adds 3 new configuration settings to the MailProxy Options Tab, for
> comfortably enabling greylisting for IPv6 addresses, and configuring the
> netmask used for greylist matching
> 
> Signed-off-by: Stoiko Ivanov <s.ivanov at proxmox.com>
> ---
>   js/MailProxyOptions.js | 11 ++++++++++-
>   1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/js/MailProxyOptions.js b/js/MailProxyOptions.js
> index 58a6a8a..bc4f3f6 100644
> --- a/js/MailProxyOptions.js
> +++ b/js/MailProxyOptions.js
> @@ -44,9 +44,18 @@ Ext.define('PMG.MailProxyOptions', {
>   	});
>   	/*jslint confusion: false*/
>   
> -	me.add_boolean_row('greylist', gettext('Use Greylisting'),
> +	me.add_boolean_row('greylist', gettext('Use Greylisting for IPv4'),
>   			   { defaultValue: 1 });
>   
> +	me.add_integer_row('greylistmask4', gettext('Netmask for Greylisting IPv4'),
> +			   { defaultValue: 24, minValue: 0 });
> +
> +	me.add_boolean_row('greylist6', gettext('Use Greylisting for IPv6'),
> +			   { defaultValue: 0 });
> +
> +	me.add_integer_row('greylistmask6', gettext('Netmask for Greylisting IPv6'),
> +			   { defaultValue: 64, minValue: 0 });

we can add also a maxValue here, so the users cannot put a 130 subnet in 
and get an api error

> +
>   	me.add_boolean_row('spf', gettext('Use SPF'), { defaultValue: 1 });
>   
>   	me.add_boolean_row('hide_received', gettext('Hide Internal Hosts'));
>