From: spirillen <44526987+spirillen@users.noreply.github.com>
Date: Wed, 5 Dec 2018 12:05:42 +0000 (+0100)
Subject: Optimizing and more descriptive rpzMaster ip array
X-Git-Tag: rec-4.2.0-alpha1~46^2~2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=1fe73ff7e09c2a3146b9aad03cedf28eacf16db3;p=thirdparty%2Fpdns.git

Optimizing and more descriptive rpzMaster ip array

As I found my own "old" description of the multiply ip array in rpzMaster rather unfriendly and less descriptive I'm trying to make this easier to read and adding a more descriptive text.

Coal: make it more descriptive, understandable and avoid language misunderstanding.

Please comment and give suggestion for optimization

/Spirillen
---

diff --git a/pdns/recursordist/docs/lua-config/rpz.rst b/pdns/recursordist/docs/lua-config/rpz.rst
index 80cda4bdec..5d37f7e504 100644
--- a/pdns/recursordist/docs/lua-config/rpz.rst
+++ b/pdns/recursordist/docs/lua-config/rpz.rst
@@ -28,10 +28,14 @@ To slave from a master and start IXFR to get updates, use for example:
 
 In this example, 'policy.rpz' denotes the name of the zone to query for.
 
-As of .. versionchanged:: 4.2.0: you can add IP blocks into the rpzMaster as:
-
-    rpzMaster({"192.0.2.4","192.0.2.5"}, "policy.rpz", {defpol=Policy.Drop})
-    
+New in version 4.2.0: You can add several IP blocks into the :func:`rpzMaster` for backends failover and load balancing per zones as an comma seperated array with ip addresses enclosed in "" (double quotes) and add optional port:
+
+    rpzMaster({"192.0.2.4","192.0.2.5:5301"}, "policy.rpz", {defpol=Policy.Drop})
+  
+  In the above example we have truncated two rpzMaster lines into one line by using array and added optional port number 5301.
+  (If no optional port is set, the default port 53 is used)
+  
+   
 .. function:: rpzFile(filename, settings)
 
   Load an RPZ from disk.