[MDL-49707] Enforce maximum length for localpart in message_inbound configuration - Moodle Tracker

XML

Word

Printable

Details

Type: Bug
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 2.8, 2.8.6, 2.9
Fix Version/s: 2.8.7, 2.9.1
Component/s: Administration
Labels:
- ci
- regression
- triaged

Testing Instructions:
Hide

Go to something like http://localhost/stable_master_forum/admin/search.php?query=inbound

Try submitting a value for messageinbound_mailbox, that is more than 15 chars and make sure you get an error.

Submit a value less than 15 chars and make sure it is saved as expected.
Show
Go to something like http://localhost/stable_master_forum/admin/search.php?query=inbound Try submitting a value for messageinbound_mailbox, that is more than 15 chars and make sure you get an error. Submit a value less than 15 chars and make sure it is saved as expected.
Affected Branches:
MOODLE_28_STABLE, MOODLE_29_STABLE
Fixed Branches:
MOODLE_28_STABLE, MOODLE_29_STABLE
Pull from Repository:
git://github.com/ankitagarwal/moodle.git
Pull Master Branch:
~~MDL-49707~~-master
Pull Master Diff URL:
https://github.com/ankitagarwal/moodle/compare/803f5657...MDL-49707-master

Description

As noted in https://moodle.org/mod/forum/discuss.php?d=277594#p1244223, The message_inbound configuration is missing a check on the length of the localpart section.

As per the RFC, the localpart of an e-mail address must be no greater than 64 characters including any subaddress component. At the moment, this is not enforced, nor is it checked. If a longer address is specified, it is generated, but not used causing much confusion and consternation.

Attachments

Issue Links

is duplicated by

MDL-49610 The mailbox name in the Icoming mail configuration is not validated

Closed

Activity

People

Assignee:: Ankit Agarwal

Reporter:: Andrew Nicols

Peer reviewer:: Mark Nelson

Integrator:: Dan Poltawski

Tester:: Adrian Greeve

Participants:: Adrian Greeve, Andrew Nicols, Ankit Agarwal, CiBoT, Dan Poltawski, Eloy Lafuente (stronk7), Mark Nelson

Component watchers:: Andrew Nicols, Jun Pataleta, Michael Hawkins, Shamim Rezaie, Simey Lameze

Votes:: 1 Vote for this issue

Watchers:: 3 Start watching this issue

Dates

Created:: 30/Mar/15 5:40 PM

Updated:: 22/Oct/15 11:42 PM

Resolved:: 05/Jun/15 6:50 AM

Fix Release Date:: 6/Jul/15