hcq/MaxScale
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

MXS-2329 Allow the restriction of duration units

Browse Source 
It's now possible to specify in the config parameter declaration
that the smallest allowed unit is seconds. For parameters whose
granularity is seconds, allowing to specify a duration in
milliseconds would open up a possibility for hard to detect errors.
This commit is contained in:
Johan Wikman
2019-04-29 10:38:25 +03:00
parent 8a250a8b13
commit 638debcdc0
3 changed files with 18 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
server/core/config.cc
View File

@ -4484,13 +4484,27 @@ bool config_param_is_valid(const MXS_MODULE_PARAM* params,
{
valid = true;
if (unit == mxs::config::DURATION_IN_DEFAULT)
switch (unit)
{
case mxs::config::DURATION_IN_MILLISECONDS:
if (params[i].options & MXS_MODULE_OPT_DURATION_S)
{
MXS_ERROR("Currently the granularity of '%s' is seconds. The value "
"cannot be specified in milliseconds.", params[i].name);
valid = false;
}
break;
case mxs::config::DURATION_IN_DEFAULT:
MXS_WARNING("Specifying durations without a suffix denoting the unit "
"has been deprecated: '%s=%s'. Use the suffixes 'h' (hour), "
"'m' (minute) 's' (second) or 'ms' (milliseconds). "
"For instance, '%s=%ss' or '%s=%sms.",
key, value, key, value, key, value);
break;
default:
break;
}
}
}

2
server/core/test/test_config.cc
View File

@ -44,6 +44,7 @@ int test_validity()
{"p7", MXS_MODULE_PARAM_SERVICE, "my-service" },
{"p8", MXS_MODULE_PARAM_ENUM, "a", MXS_MODULE_OPT_ENUM_UNIQUE, enum_values},
{"p9", MXS_MODULE_PARAM_DURATION, "4711s" },
{"p10", MXS_MODULE_PARAM_DURATION, "4711s", MXS_MODULE_OPT_DURATION_S },
{MXS_END_MODULE_PARAMS}
};
@ -107,6 +108,7 @@ int test_validity()
TEST(config_param_is_valid(params, "p9", "4711S", &ctx));
TEST(config_param_is_valid(params, "p9", "4711MS", &ctx));
TEST(!config_param_is_valid(params, "p9", "4711q", &ctx));
TEST(!config_param_is_valid(params, "p10", "4711ms", &ctx));
/** Service parameter */
CONFIG_CONTEXT svc("test-service");