Project

General

Profile

Actions

Bug #17036

closed

Still 250000 bytes maximum size is hardcoded for mailform attachments

Added by Jo Hasenau over 17 years ago. Updated over 14 years ago.

Status:
Closed
Priority:
Should have
Category:
Communication
Target version:
-
Start date:
2007-02-24
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.2
PHP Version:
4.3
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

People using the integrated mailform functions can't create forms, where users would be able to send attachments larger than 250000 bytes. This is really bad if you are using it i.e. as a contact form like this:
http://www.xposign.com/20.0.html
People can send large files of plans and maps here which is IMHO not very uncommon.

The value is hardcoded in class.t3lib_formmail.php, ahs been raised to 250000 a few versions ago but instead it should be replaced by a TS-config or TS-Setup parameter. The default could still be 250000 but it would be possible to raise this to a higher maximum without having to hack a core file each time you updated the source.

(issue imported from #M5065)


Files

5065_config_default.diff (2.29 KB) 5065_config_default.diff Administrator Admin, 2007-03-10 01:09
5060_class.t3lib_formmail.php.diff (574 Bytes) 5060_class.t3lib_formmail.php.diff Administrator Admin, 2007-03-10 01:09

Related issues 2 (0 open2 closed)

Has duplicate TYPO3 Core - Feature #16338: hardcoded limit for mailform attachmentClosedOliver Hader2006-07-11

Actions
Has duplicate TYPO3 Core - Bug #15220: Filesize of maximum allowed files in mailformsClosedMartin Kutschker2005-11-18

Actions
Actions #1

Updated by Oliver Hader over 17 years ago

+1 for having a higher default value and making this configurable globally and/or with TSconfig.

Actions #2

Updated by Martin Kutschker over 17 years ago

You cannot set a vonfig for a t3lib-thingy per TS directly, but there could a) be a global default (localconf.php/installer) and b) a TS setting for FORM cObjects.

Actions #3

Updated by Jo Hasenau over 17 years ago

sounds good to me.

So a) would be the must have and b) the nice to have

;-)

Actions #4

Updated by Steffen Kamper over 17 years ago

please make it with a variable, e.g.

$TYPO3_CONF_VARS['FE']['maxMailAttachmentSize']=250000;

So everyone can configure how he likes.

Actions #5

Updated by Christian Hernmarck over 17 years ago

This is a long time "bug" (maybe not here in the buglist)

2005-11-30 (german):
http://blog.bluehands.de/blogs/index.php?blog=6&title=typo3_form_mail_attachment_max_size&more=1&c=1&tb=1&pb=1

so, it's not only in 4.1rcX...

Hope, this will be "fixed" in upcoming versions, 4.0.6 and 4.1...

Actions #6

Updated by Martin Kutschker over 17 years ago

Adding a config variable is a new feature, so it won't make it in to 4.0.x and as 4.1 went out only yesterday, you'll have to wait for 4.2.

Actions #7

Updated by Steffen Kamper over 17 years ago

ok, i take the chance and make a feature request for 4.2 :-)

i added a config-var and set the default a little higher to 400.000:
$TYPO3_CONF_VARS['FE']['maxMailAttachmentSize']=400000;
Then i replaced the hardcoded value with the var.

patch is included.

Actions #8

Updated by Ernesto Baschny almost 17 years ago

Patch in core list, to be considered for 4.2.

Actions #10

Updated by Ernesto Baschny almost 17 years ago

Commited to trunk (rev 2849)

Actions

Also available in: Atom PDF