Skip to content

Commit 19ca88a

Browse files
committed
zcore: delete config directive "max_mail_num"
Administratively limiting the global message count per store can be done in exmdb; there should be no need to (re-)do an additional check within zcore.
1 parent 05cce02 commit 19ca88a

File tree

5 files changed

+5
-20
lines changed

5 files changed

+5
-20
lines changed

doc/zcore.8gx

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,9 +86,6 @@ Default: \fI64M\fP
8686
\fBmax_ext_rule_length\fP
8787
Default: \fI510K\fP
8888
.TP
89-
\fBmax_mail_num\fP
90-
Default: \fI1000000\fP
91-
.TP
9289
\fBmax_rcpt_num\fP
9390
The maximum number of recipients that an e-mail is allowed to have.
9491
.br

exch/zcore/common_util.cpp

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ struct LANGMAP_ITEM {
8282
}
8383

8484
size_t g_max_mail_len;
85-
unsigned int g_max_rcpt, g_max_message;
85+
unsigned int g_max_rcpt;
8686
unsigned int g_max_rule_len, g_max_extrule_len, zcore_backfill_transporthdr;
8787
static std::string g_smtp_url;
8888
char g_org_name[256];
@@ -329,13 +329,12 @@ BOOL common_util_exmdb_locinfo_from_string(
329329
}
330330

331331
void common_util_init(const char *org_name, const char *default_charset,
332-
unsigned int max_rcpt, unsigned int max_message, size_t max_mail_len,
332+
unsigned int max_rcpt, size_t max_mail_len,
333333
unsigned int max_rule_len, std::string &&smtp_url, const char *submit_command)
334334
{
335335
gx_strlcpy(g_org_name, org_name, std::size(g_org_name));
336336
gx_strlcpy(g_default_charset, default_charset, std::size(g_default_charset));
337337
g_max_rcpt = max_rcpt;
338-
g_max_message = max_message;
339338
g_max_mail_len = max_mail_len;
340339
g_max_rule_len = g_max_extrule_len = max_rule_len;
341340
g_smtp_url = std::move(smtp_url);

exch/zcore/common_util.hpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ struct message_content;
7878
struct message_object;
7979
struct store_object;
8080

81-
extern void common_util_init(const char *org_name, const char *default_charset, unsigned int max_rcpt, unsigned int max_msg, size_t max_mail_len, unsigned int max_rule_len, std::string &&smtp_url, const char *submit_cmd);
81+
extern void common_util_init(const char *org_name, const char *default_charset, unsigned int max_rcpt, size_t max_mail_len, unsigned int max_rule_len, std::string &&smtp_url, const char *submit_cmd);
8282
extern int common_util_run(const char *data_path);
8383
BOOL common_util_verify_columns_and_sorts(
8484
const PROPTAG_ARRAY *pcolumns,
@@ -162,7 +162,7 @@ extern ec_error_t cu_fbdata_to_ical(const char *, const char *, time_t, time_t,
162162
extern bool permrow_entryids_equal(const PERMISSION_ROW &, const uint32_t *, const BINARY *);
163163

164164
extern size_t g_max_mail_len;
165-
extern unsigned int g_max_rcpt, g_max_message;
165+
extern unsigned int g_max_rcpt;
166166
extern unsigned int g_max_rule_len, g_max_extrule_len;
167167
extern unsigned int zcore_backfill_transporthdr;
168168
extern char g_org_name[256];

exch/zcore/main.cpp

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,6 @@ static constexpr cfg_directive zcore_cfg_defaults[] = {
9797
{"mail_max_length", "64M", CFG_SIZE, "1"},
9898
{"mailbox_ping_interval", "5min", CFG_TIME, "1min", "1h"},
9999
{"max_ext_rule_length", "510K", CFG_SIZE, "1"},
100-
{"max_mail_num", "1000000", CFG_SIZE, "1"},
101100
{"max_rcpt_num", "256", CFG_SIZE, "1"},
102101
{"notify_stub_threads_num", "10", CFG_SIZE, "1", "100"},
103102
{"oxcical_allday_ymd", "1", CFG_BOOL},
@@ -297,9 +296,6 @@ int main(int argc, char **argv)
297296
auto max_rcpt = pconfig->get_ll("max_rcpt_num");
298297
mlog(LV_INFO, "system: maximum rcpt number is %lld", max_rcpt);
299298

300-
auto max_mail = pconfig->get_ll("max_mail_num");
301-
mlog(LV_INFO, "system: maximum mail number is %lld", max_mail);
302-
303299
auto max_length = pconfig->get_ll("mail_max_length");
304300
HX_unit_size(temp_buff, std::size(temp_buff), max_length, 1024, 0);
305301
mlog(LV_INFO, "system: maximum mail length is %s", temp_buff);
@@ -321,7 +317,7 @@ int main(int argc, char **argv)
321317

322318
common_util_init(g_config_file->get_value("x500_org_name"),
323319
g_config_file->get_value("default_charset"),
324-
max_rcpt, max_mail, max_length, max_rule_len, std::move(smtp_url),
320+
max_rcpt, max_length, max_rule_len, std::move(smtp_url),
325321
g_config_file->get_value("submit_command"));
326322

327323
int proxy_num = pconfig->get_ll("rpc_proxy_connection_num");

exch/zcore/zserver.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1561,13 +1561,6 @@ ec_error_t zs_createmessage(GUID hsession,
15611561
uint64_t total_size = lnum != nullptr ? *lnum : 0;
15621562
if (max_quota > 0 && total_size > static_cast<uint64_t>(max_quota))
15631563
return ecQuotaExceeded;
1564-
num = tmp_propvals.get<uint32_t>(PR_ASSOC_CONTENT_COUNT);
1565-
uint32_t total_mail = num != nullptr ? *num : 0;
1566-
num = tmp_propvals.get<uint32_t>(PR_CONTENT_COUNT);
1567-
if (num != nullptr)
1568-
total_mail += *num;
1569-
if (total_mail > g_max_message)
1570-
return ecQuotaExceeded;
15711564
if (!exmdb_client->allocate_message_id(pstore->get_dir(),
15721565
folder_id, &message_id))
15731566
return ecError;

0 commit comments

Comments
 (0)