]> git.ipfire.org Git - thirdparty/dovecot/core.git/commitdiff
auth: Remove legacy passdb init and passdb_args setting
authorTimo Sirainen <timo.sirainen@open-xchange.com>
Tue, 26 Nov 2024 14:00:00 +0000 (16:00 +0200)
committerAki Tuomi <aki.tuomi@open-xchange.com>
Fri, 17 Jan 2025 08:40:01 +0000 (10:40 +0200)
src/auth/auth-settings.c
src/auth/auth-settings.h
src/auth/passdb.c
src/auth/passdb.h
src/auth/test-mock.c

index d0ea898b48dddb061de3001ef48de08e6ef5be4c..5b373bf753373a7022021576650e8cc58ff59ea4 100644 (file)
@@ -106,7 +106,6 @@ const struct setting_keyvalue auth_worker_service_settings_defaults[] = {
 static const struct setting_define auth_passdb_setting_defines[] = {
        DEF(STR, name),
        DEF(STR, driver),
-       DEF(STR_NOVARS, args),
        DEF(BOOL, fields_import_all),
        DEF(BOOLLIST, mechanisms_filter),
        DEF(STR, username_filter),
@@ -129,7 +128,6 @@ static const struct setting_define auth_passdb_setting_defines[] = {
 static const struct auth_passdb_settings auth_passdb_default_settings = {
        .name = "",
        .driver = "",
-       .args = "",
        .fields_import_all = TRUE,
        .mechanisms_filter = ARRAY_INIT,
        .username_filter = "",
index 3cc521a7211d590d930d1a2b761210c259872250..a49921484b374bc0696a334e19c324dadd318338 100644 (file)
@@ -13,7 +13,6 @@ struct auth_passdb_settings {
        pool_t pool;
        const char *name;
        const char *driver;
-       const char *args;
        bool fields_import_all;
        ARRAY_TYPE(const_string) mechanisms_filter;
        const char *username_filter;
index 5d9602ed99ec04759ca86500ae703a8cd3ae0aa8..0dd962813151af0cd3c7a0d4da2626da8ced2ac2 100644 (file)
@@ -185,29 +185,18 @@ passdb_preinit(pool_t pool, struct event *event,
                i_fatal("Support not compiled in for passdb driver '%s'",
                        set->driver);
        }
-       if (iface->preinit_legacy == NULL && iface->init == NULL &&
-           *set->args != '\0') {
-               i_fatal("passdb %s: No args are supported: %s",
-                       set->driver, set->args);
-       }
 
        if (iface->preinit != NULL) {
-               if (set->args[0] != '\0')
-                       i_fatal("passdb %s: passdb_args must be empty", set->name);
                if (iface->preinit(pool, event, &passdb, &error) < 0)
                        i_fatal("passdb %s: %s", set->name, error);
                passdb->default_pass_scheme =
                        set->default_password_scheme;
                passdb->blocking = set->use_worker;
        } else {
-               if (iface->preinit_legacy == NULL)
-                       passdb = p_new(pool, struct passdb_module, 1);
-               else
-                       passdb = iface->preinit_legacy(pool, set->args);
+               passdb = p_new(pool, struct passdb_module, 1);
        }
        passdb->id = ++auth_passdb_id;
        passdb->iface = *iface;
-       passdb->args = p_strdup(pool, set->args);
        array_push_back(&passdb_modules, &passdb);
        return passdb;
 }
index 0404303a659da495314c3a6ca123a104f56b5876..102df34c56a4b50c943932e0c60dd72a1772f61e 100644 (file)
@@ -39,7 +39,6 @@ struct passdb_module_interface {
           given event. */
        int (*preinit)(pool_t pool, struct event *event,
                       struct passdb_module **module_r, const char **error_r);
-       struct passdb_module *(*preinit_legacy)(pool_t pool, const char *args);
        void (*init)(struct passdb_module *module);
        void (*deinit)(struct passdb_module *module);
 
@@ -59,7 +58,6 @@ struct passdb_module_interface {
 };
 
 struct passdb_module {
-       const char *args;
        /* The default caching key for this module, or NULL if caching isn't
           wanted. This is updated by settings in auth_passdb. */
        const char *default_cache_key;
index d0e3562421920ab646108e783ad2abc65614d698..9a791e9c50affe514bf7ecff74f5a17e39f8ed75 100644 (file)
@@ -55,7 +55,6 @@ void passdb_mock_mod_init(void)
        struct auth_passdb_settings set = {
                .name = "mock",
                .driver = "mock",
-               .args = "",
                .mechanisms_filter = ARRAY_INIT,
                .username_filter = "",