On Fri, Jan 10, 2014 at 05:32:00PM +0100, Pavel Reichl wrote:
Hi Jakub,
thank you for comments. New patches addressing issues are attached.
PR
--- a/src/providers/simple/simple_access.c
+++ b/src/providers/simple/simple_access.c
@@ -32,7 +32,76 @@
#define CONFDB_SIMPLE_ALLOW_GROUPS "simple_allow_groups"
#define CONFDB_SIMPLE_DENY_GROUPS "simple_deny_groups"
The code looks good and works well. I only have one nitpick. Can you
rename this constant to not include the word 'PERIOD'. That sounds like
a background task which would run on its own. "Timeout" might be more
suitable.
+#define PERIOD_OF_REFRESH_FILTER_LISTS 5
+