Hi, Sergey! On Nov 27, Sergey Vojtovich wrote: > > And what I always hated in this validation stuff is that you'll never > come up with a password that makes it happy. Of course unless you know > requirements. In our case requirements are exposed via system > variables. This is more or less acceptable.
In simple validation plugins requirements are, well, simple and one can look at system variables. In the cracklib case the warning reports the exact cracklib error, like in: Warning 1819 cracklib: it does not contain enough DIFFERENT characters or Warning 1819 cracklib: it is based on your username or Warning 1819 cracklib: it is based on a dictionary word I can add a similar warning for the simple plugin too, if needed (I didn't think it was needed, though). > > diff --git a/sql/sql_plugin.cc b/sql/sql_plugin.cc > > index 529a795..cdf8c7b 100644 > > --- a/sql/sql_plugin.cc > > +++ b/sql/sql_plugin.cc > > @@ -1051,7 +1055,8 @@ static bool plugin_add(MEM_ROOT *tmp_root, > > continue; // invalid plugin type > > > > if (plugin->type == MYSQL_UDF_PLUGIN || > > - (plugin->type == 8 && tmp.plugin_dl->mariaversion == 0)) > > + (plugin->type == MariaDB_PASSWORD_VALIDATION_INTERFACE_VERSION && > > + tmp.plugin_dl->mariaversion == 0)) > > continue; // unsupported plugin type > > > > if (name->str && my_strnncoll(system_charset_info, > You compare type against version here. Why? Typo, sorry. Fixed. Regards, Sergei _______________________________________________ Mailing list: https://launchpad.net/~maria-developers Post to : [email protected] Unsubscribe : https://launchpad.net/~maria-developers More help : https://help.launchpad.net/ListHelp
