X-Git-Url: https://code.citadel.org/?a=blobdiff_plain;f=textclient%2Ftuiconfig.c;h=28a10ed15748772d5808a44bde2da45c30949b42;hb=HEAD;hp=48e009750acb22fad6966044455ae5bd103cc8a1;hpb=da6bb5e24252a0ff56d314a60081ae9ed31ab1a7;p=citadel.git diff --git a/textclient/tuiconfig.c b/textclient/tuiconfig.c index 48e009750..28a10ed15 100644 --- a/textclient/tuiconfig.c +++ b/textclient/tuiconfig.c @@ -1,16 +1,8 @@ -/* - * Configuration screens that are part of the text mode client. - * - * Copyright (c) 1987-2018 by the citadel.org team - * - * This program is open source software; you can redistribute it and/or modify - * it under the terms of the GNU General Public License version 3. - * - * This program is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU General Public License for more details. - */ +// Configuration screens that are part of the text mode client. +// +// Copyright (c) 1987-2022 by the citadel.org team +// +// This program is open source software. Use, duplication, or disclosure is subject to the GNU General Public License version 3. #include "textclient.h" @@ -24,14 +16,8 @@ extern int screenwidth; char editor_path[PATH_MAX]; -/* - * General system configuration command - */ -void do_system_configuration(CtdlIPC * ipc) -{ - - /* NUM_CONFIGS is now defined in citadel.h */ - +// General system configuration command +void do_system_configuration(CtdlIPC * ipc) { char buf[256]; char sc[NUM_CONFIGS][256]; char *resp = NULL; @@ -39,13 +25,13 @@ void do_system_configuration(CtdlIPC * ipc) struct ExpirePolicy *mbx_expirepolicy = NULL; int a; int logpages = 0; - int r; /* IPC response code */ + int r; // IPC response code int server_configs = 0; - /* Clear out the config buffers */ + // Clear out the config buffers memset(&sc[0][0], 0, sizeof(sc)); - /* Fetch the current config */ + // Fetch the current config r = CtdlIPCGetSystemConfig(ipc, &resp, buf); if (r / 100 == 1) { server_configs = num_tokens(resp, '\n'); @@ -55,17 +41,16 @@ void do_system_configuration(CtdlIPC * ipc) } } } - if (resp) + if (resp) { free(resp); + } resp = NULL; - /* Fetch the expire policy (this will silently fail on old servers, - * resulting in "default" policy) - */ + + // Fetch the expire policy (this will silently fail on old servers, resulting in "default" policy) r = CtdlIPCGetMessageExpirationPolicy(ipc, 2, &site_expirepolicy, buf); r = CtdlIPCGetMessageExpirationPolicy(ipc, 3, &mbx_expirepolicy, buf); - /* Identification parameters */ - + // Identification parameters strprompt("Node name", &sc[0][0], 15); strprompt("Fully qualified domain name", &sc[1][0], 63); strprompt("Human readable node name", &sc[2][0], 20); @@ -74,52 +59,41 @@ void do_system_configuration(CtdlIPC * ipc) strprompt("Name of system administrator", &sc[13][0], 25); strprompt("Paginator prompt", &sc[10][0], 79); - /* Security parameters */ - + // Security parameters snprintf(sc[7], sizeof sc[7], "%d", (boolprompt("Require registration for new users", atoi(&sc[7][0])))); snprintf(sc[29], sizeof sc[29], "%d", (boolprompt("Disable self-service user account creation", atoi(&sc[29][0])))); strprompt("Initial access level for new users", &sc[6][0], 1); strprompt("Access level required to create rooms", &sc[19][0], 1); snprintf(sc[67], sizeof sc[67], "%d", (boolprompt("Allow anonymous guest logins", atoi(&sc[67][0])))); - snprintf(sc[4], sizeof sc[4], "%d", (boolprompt("Automatically give room admin privs to a user who creates a private room", - atoi(&sc[4][0])))); - + snprintf(sc[4], sizeof sc[4], "%d", + (boolprompt("Automatically give room admin privs to a user who creates a private room", atoi(&sc[4][0])))); snprintf(sc[8], sizeof sc[8], "%d", (boolprompt("Automatically move problem user messages to twit room", atoi(&sc[8][0])))); - strprompt("Name of twit room", &sc[9][0], ROOMNAMELEN); - snprintf(sc[11], sizeof sc[11], "%d", (boolprompt("Restrict Internet mail to only those with that privilege", - atoi(&sc[11][0])))); + snprintf(sc[11], sizeof sc[11], "%d", + (boolprompt("Restrict Internet mail to only those with that privilege", atoi(&sc[11][0])))); snprintf(sc[26], sizeof sc[26], "%d", (boolprompt("Allow admins to Zap (forget) rooms", atoi(&sc[26][0])))); if (!IsEmptyStr(&sc[18][0])) { logpages = 1; - } else { + } + else { logpages = 0; } logpages = boolprompt("Log all instant messages", logpages); if (logpages) { strprompt("Name of logging room", &sc[18][0], ROOMNAMELEN); - } else { + } + else { sc[18][0] = 0; } - /* Commented out because this setting isn't really appropriate to - * change while the server is running. - * - * snprintf(sc[52], sizeof sc[52], "%d", (boolprompt( - * "Use system authentication", - * atoi(&sc[52][0])))); - */ - - /* Server tuning */ - + // Server tuning strprompt("Server connection idle timeout (in seconds)", &sc[5][0], 4); strprompt("Maximum concurrent sessions", &sc[14][0], 4); strprompt("Maximum message length", &sc[20][0], 20); strprompt("Minimum number of worker threads", &sc[21][0], 3); strprompt("Maximum number of worker threads", &sc[22][0], 3); snprintf(sc[43], sizeof sc[43], "%d", (boolprompt("Automatically delete committed database logs", atoi(&sc[43][0])))); - strprompt("Server IP address (* for 'any')", &sc[37][0], 15); strprompt("POP3 server port (-1 to disable)", &sc[23][0], 5); strprompt("POP3S server port (-1 to disable)", &sc[40][0], 5); @@ -128,18 +102,15 @@ void do_system_configuration(CtdlIPC * ipc) strprompt("SMTP MTA server port (-1 to disable)", &sc[24][0], 5); strprompt("SMTP MSA server port (-1 to disable)", &sc[38][0], 5); strprompt("SMTPS server port (-1 to disable)", &sc[41][0], 5); + snprintf(sc[72], sizeof sc[72], "%d", (boolprompt("Advertise STARTTLS on the SMTP port", atoi(&sc[72][0])))); strprompt("NNTP server port (-1 to disable)", &sc[70][0], 5); strprompt("NNTPS server port (-1 to disable)", &sc[71][0], 5); strprompt("Postfix TCP Dictionary Port server port (-1 to disable)", &sc[50][0], 5); strprompt("ManageSieve server port (-1 to disable)", &sc[51][0], 5); - strprompt("XMPP (Jabber) client to server port (-1 to disable)", &sc[62][0], 5); - /* No prompt because we don't implement this service yet, it's just a placeholder */ - /* strprompt("XMPP (Jabber) server to server port (-1 to disable)", &sc[63][0], 5); */ + // strprompt("XMPP (Jabber) server to server port (-1 to disable)", &sc[63][0], 5); This is just a placeholder. - /* This logic flips the question around, because it's one of those - * situations where 0=yes and 1=no - */ + // This logic flips the question around, because it's one of those situations where 0=yes and 1=no a = atoi(sc[25]); a = (a ? 0 : 1); a = boolprompt("Correct forged From: lines during authenticated SMTP", a); @@ -148,24 +119,22 @@ void do_system_configuration(CtdlIPC * ipc) snprintf(sc[66], sizeof sc[66], "%d", (boolprompt("Flag messages as spam instead of rejecting", atoi(&sc[66][0])))); - /* This logic flips the question around, because it's one of those - * situations where 0=yes and 1=no - */ + // This logic flips the question around, because it's one of those situations where 0=yes and 1=no a = atoi(sc[61]); a = (a ? 0 : 1); a = boolprompt("Force IMAP posts in public rooms to be from the user who submitted them", a); a = (a ? 0 : 1); snprintf(sc[61], sizeof sc[61], "%d", a); - snprintf(sc[45], sizeof sc[45], "%d", (boolprompt("Allow unauthenticated SMTP clients to spoof my domains", - atoi(&sc[45][0])))); - snprintf(sc[57], sizeof sc[57], "%d", (boolprompt("Perform RBL checks at greeting instead of after RCPT", - atoi(&sc[57][0])))); + snprintf(sc[45], sizeof sc[45], "%d", + (boolprompt("Allow unauthenticated SMTP clients to spoof my domains", atoi(&sc[45][0])))); + snprintf(sc[57], sizeof sc[57], "%d", + (boolprompt("Perform RBL checks at greeting instead of after RCPT", atoi(&sc[57][0])))); - /* LDAP settings */ + // LDAP settings if (ipc->ServInfo.supports_ldap) { a = strlen(&sc[32][0]); - a = (a ? 1 : 0); /* Set only to 1 or 0 */ + a = (a ? 1 : 0); // Set only to 1 or 0 a = boolprompt("Do you want to configure LDAP authentication?", a); if (a) { strprompt("Host name of LDAP server", &sc[32][0], 127); @@ -173,16 +142,17 @@ void do_system_configuration(CtdlIPC * ipc) strprompt("Base DN", &sc[34][0], 255); strprompt("Bind DN (or blank for anonymous bind)", &sc[35][0], 255); strprompt("Password for bind DN (or blank for anonymous bind)", &sc[36][0], 255); - } else { + } + else { strcpy(&sc[32][0], ""); } } - /* Expiry settings */ + // Expiry settings strprompt("Default user purge time (days)", &sc[16][0], 5); strprompt("Default room purge time (days)", &sc[17][0], 5); - /* Angels and demons dancing in my head... */ + // Angels and demons dancing in my head... do { snprintf(buf, sizeof buf, "%d", site_expirepolicy->expire_mode); strprompt("System default message expire policy (? for list)", buf, 1); @@ -194,7 +164,7 @@ void do_system_configuration(CtdlIPC * ipc) } while ((buf[0] < '1') || (buf[0] > '3')); site_expirepolicy->expire_mode = buf[0] - '0'; - /* ...lunatics and monsters underneath my bed */ + // ...lunatics and monsters underneath my bed if (site_expirepolicy->expire_mode == 2) { snprintf(buf, sizeof buf, "%d", site_expirepolicy->expire_value); strprompt("Keep how many messages online?", buf, 10); @@ -206,7 +176,7 @@ void do_system_configuration(CtdlIPC * ipc) site_expirepolicy->expire_value = atol(buf); } - /* Media messiahs preying on my fears... */ + // Media messiahs preying on my fears... do { snprintf(buf, sizeof buf, "%d", mbx_expirepolicy->expire_mode); strprompt("Mailbox default message expire policy (? for list)", buf, 1); @@ -219,7 +189,7 @@ void do_system_configuration(CtdlIPC * ipc) } while ((buf[0] < '0') || (buf[0] > '3')); mbx_expirepolicy->expire_mode = buf[0] - '0'; - /* ...Pop culture prophets playing in my ears */ + // ...Pop culture prophets playing in my ears if (mbx_expirepolicy->expire_mode == 2) { snprintf(buf, sizeof buf, "%d", mbx_expirepolicy->expire_value); strprompt("Keep how many messages online?", buf, 10); @@ -235,46 +205,34 @@ void do_system_configuration(CtdlIPC * ipc) strprompt("Default frequency to run POP3 collection (in seconds)", &sc[64][0], 5); strprompt("Fastest frequency to run POP3 collection (in seconds)", &sc[65][0], 5); strprompt("Hour to run purges (0-23)", &sc[31][0], 2); - snprintf(sc[42], sizeof sc[42], "%d", (boolprompt("Enable full text search index (warning: resource intensive)", - atoi(&sc[42][0])))); - + snprintf(sc[42], sizeof sc[42], "%d", + (boolprompt("Enable full text search index (warning: resource intensive)", atoi(&sc[42][0])))); snprintf(sc[46], sizeof sc[46], "%d", (boolprompt("Perform journaling of email messages", atoi(&sc[46][0])))); snprintf(sc[47], sizeof sc[47], "%d", (boolprompt("Perform journaling of non-email messages", atoi(&sc[47][0])))); if ((atoi(&sc[46][0])) || (atoi(&sc[47][0]))) { strprompt("Email destination of journalized messages", &sc[48][0], 127); } - /* No more Funambol */ + // No more Funambol sc[53][0] = 0; sc[54][0] = 0; sc[55][0] = 0; sc[56][0] = 0; - /* External pager stuff */ + // External pager stuff int yes_pager = 0; - if (strlen(sc[60]) > 0) + if (strlen(sc[60]) > 0) { yes_pager = 1; + } yes_pager = boolprompt("Configure an external pager tool", yes_pager); if (yes_pager) { strprompt("External pager tool", &sc[60][0], 255); - } else { - sc[60][0] = 0; } - - /* Master user account */ - int yes_muacct = 0; - if (strlen(sc[58]) > 0) - yes_muacct = 1; - yes_muacct = boolprompt("Enable a 'master user' account", yes_muacct); - if (yes_muacct) { - strprompt("Master user name", &sc[58][0], 31); - strprompt("Master user password", &sc[59][0], -31); - } else { - strcpy(&sc[58][0], ""); - strcpy(&sc[59][0], "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"); + else { + sc[60][0] = 0; } - /* Save it */ + // Save it scr_printf("Save this configuration? "); if (yesno()) { r = 1; @@ -314,11 +272,8 @@ void do_system_configuration(CtdlIPC * ipc) } -/* - * support function for do_internet_configuration() - */ -void get_inet_rec_type(CtdlIPC * ipc, char *buf) -{ +// support function for do_internet_configuration() +void get_inet_rec_type(CtdlIPC * ipc, char *buf) { int sel; keyopt(" <1> localhost (Alias for this computer)\n"); @@ -355,11 +310,8 @@ void get_inet_rec_type(CtdlIPC * ipc, char *buf) } -/* - * Internet mail configuration - */ -void do_internet_configuration(CtdlIPC * ipc) -{ +// Internet mail configuration +void do_internet_configuration(CtdlIPC * ipc) { char buf[256]; char *resp = NULL; int num_recs = 0; @@ -376,7 +328,7 @@ void do_internet_configuration(CtdlIPC * ipc) extract_token(buf, resp, 0, '\n', sizeof buf); remove_token(resp, 0, '\n'); - // VILE SLEAZY HACK: replace obsolete "directory" domains with "localhost" + // "directory" is no longer used. replace it with "localhost" char *d = strstr(buf, "|directory"); if (d != NULL) { strcpy(d, "|localhost"); @@ -391,8 +343,9 @@ void do_internet_configuration(CtdlIPC * ipc) strcpy(recs[num_recs - 1], buf); } } - if (resp) + if (resp) { free(resp); + } do { scr_printf("\n"); @@ -416,12 +369,13 @@ void do_internet_configuration(CtdlIPC * ipc) switch (ch) { case 'a': newprompt("Enter host name: ", buf, 50); - striplt(buf); + string_trim(buf); if (!IsEmptyStr(buf)) { ++num_recs; if (num_recs == 1) { recs = malloc(sizeof(char *)); - } else { + } + else { recs = realloc(recs, (sizeof(char *)) * num_recs); } strcat(buf, "|"); @@ -441,8 +395,9 @@ void do_internet_configuration(CtdlIPC * ipc) break; case 's': r = 1; - for (i = 0; i < num_recs; i++) + for (i = 0; i < num_recs; i++) { r += 1 + strlen(recs[i]); + } resp = (char *) calloc(1, r); if (!resp) { scr_printf("Can't save config - out of memory!\n"); @@ -456,7 +411,8 @@ void do_internet_configuration(CtdlIPC * ipc) r = CtdlIPCSetSystemConfigByType(ipc, INTERNETCFG, resp, buf); if (r / 100 != 4) { scr_printf("%s\n", buf); - } else { + } + else { scr_printf("Wrote %d records.\n", num_recs); modified = 0; } @@ -478,12 +434,8 @@ void do_internet_configuration(CtdlIPC * ipc) } - -/* - * Edit network configuration for room sharing, mailing lists, etc. - */ -void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) -{ +// Edit network configuration for room sharing, mailing lists, etc. +void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) { char filename[PATH_MAX]; char changefile[PATH_MAX]; int e_ex_code; @@ -541,7 +493,7 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) } fclose(tempfp); - e_ex_code = 1; /* start with a failed exit code */ + e_ex_code = 1; // start with a failed exit code stty_ctdl(SB_RESTORE); editor_pid = fork(); cksum = file_checksum(filename); @@ -565,10 +517,10 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) e_ex_code = 1; } - if (e_ex_code == 0) { /* Save changes */ + if (e_ex_code == 0) { // Save changes changefp = fopen(changefile, "w"); - /* Load all netconfig entries that are *not* of the type we are editing */ + // Load all netconfig entries that are *not* of the type we are editing r = CtdlIPCGetRoomNetworkConfig(ipc, &listing, buf); if (r / 100 == 1) { while (listing && !IsEmptyStr(listing)) { @@ -585,14 +537,14 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) listing = NULL; } - /* ...and merge that with the data we just edited */ + // ...and merge that with the data we just edited tempfp = fopen(filename, "r"); while (fgets(buf, sizeof buf, tempfp) != NULL) { for (i = 0; i < strlen(buf); ++i) { if (buf[i] == '#') buf[i] = 0; } - striplt(buf); + string_trim(buf); if (!IsEmptyStr(buf)) { fprintf(changefp, "%s|%s\n", entrytype, buf); } @@ -600,7 +552,7 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) fclose(tempfp); fclose(changefp); - /* now write it to the server... */ + // now write it to the server... changefp = fopen(changefile, "r"); if (changefp != NULL) { listing = load_message_from_file(changefp); @@ -613,16 +565,13 @@ void network_config_management(CtdlIPC * ipc, char *entrytype, char *comment) } } - unlink(filename); /* Delete the temporary files */ + unlink(filename); // Delete the temporary files unlink(changefile); } -/* - * POP3 aggregation client configuration - */ -void do_pop3client_configuration(CtdlIPC * ipc) -{ +// POP3 aggregation client configuration +void do_pop3client_configuration(CtdlIPC * ipc) { char buf[SIZ]; int num_recs = 0; char **recs = NULL; @@ -688,7 +637,8 @@ void do_pop3client_configuration(CtdlIPC * ipc) ++num_recs; if (num_recs == 1) { recs = malloc(sizeof(char *)); - } else { + } + else { recs = realloc(recs, (sizeof(char *)) * num_recs); } strcpy(buf, "pop3client|"); @@ -728,7 +678,7 @@ void do_pop3client_configuration(CtdlIPC * ipc) strcat(listing, "\n"); } - /* Retrieve all the *other* records for merging */ + // Retrieve all the *other* records for merging r = CtdlIPCGetRoomNetworkConfig(ipc, &other_listing, buf); if (r / 100 == 1) { for (i = 0; i < num_tokens(other_listing, '\n'); ++i) { @@ -747,7 +697,8 @@ void do_pop3client_configuration(CtdlIPC * ipc) if (r / 100 != 4) { scr_printf("%s\n", buf); - } else { + } + else { scr_printf("Wrote %d records.\n", num_recs); modified = 0; } @@ -762,22 +713,16 @@ void do_pop3client_configuration(CtdlIPC * ipc) } while (!quitting); if (recs != NULL) { - for (i = 0; i < num_recs; ++i) + for (i = 0; i < num_recs; ++i) { free(recs[i]); + } free(recs); } } - - - - -/* - * RSS feed retrieval client configuration - */ -void do_rssclient_configuration(CtdlIPC * ipc) -{ +// RSS feed retrieval client configuration +void do_rssclient_configuration(CtdlIPC * ipc) { char buf[SIZ]; int num_recs = 0; char **recs = NULL; @@ -838,7 +783,8 @@ void do_rssclient_configuration(CtdlIPC * ipc) ++num_recs; if (num_recs == 1) { recs = malloc(sizeof(char *)); - } else { + } + else { recs = realloc(recs, (sizeof(char *)) * num_recs); } strcpy(buf, "rssclient|"); @@ -871,7 +817,7 @@ void do_rssclient_configuration(CtdlIPC * ipc) strcat(listing, "\n"); } - /* Retrieve all the *other* records for merging */ + // Retrieve all the *other* records for merging r = CtdlIPCGetRoomNetworkConfig(ipc, &other_listing, buf); if (r / 100 == 1) { for (i = 0; i < num_tokens(other_listing, '\n'); ++i) { @@ -890,7 +836,8 @@ void do_rssclient_configuration(CtdlIPC * ipc) if (r / 100 != 4) { scr_printf("%s\n", buf); - } else { + } + else { scr_printf("Wrote %d records.\n", num_recs); modified = 0; }