When handling bounce emails, some SMTP servers don't like the / character

Bug #994370 reported by Hugh Davenport
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mahara
Fix Released
Low
Hugh Davenport

Bug Description

This is present in /lib/user.php, function generate_email_processing_address, the problem occurs when a certain userid, say 38910 is used which produces a base64 encoding with a / in it.

Postfix (and other SMTP servers) don't like this in the extension part of an email address. [1]

The solution would be to change the / character for another valid email address character that isn't in the base 64 encoding.

[1] - http://www.irbs.net/internet/postfix/0501/1653.html

Revision history for this message
Hugh Davenport (hugh-davenport) wrote :
Changed in mahara:
status: New → Triaged
status: Triaged → In Progress
Revision history for this message
Mahara Bot (dev-mahara) wrote : A change has been merged

Reviewed: https://reviews.mahara.org/1186
Committed: http://gitorious.org/mahara/mahara/commit/0d0d3f70ce376a4bb9dde080c13b6a6bf7d3057d
Submitter: Richard Mansfield (<email address hidden>)
Branch: master

commit 0d0d3f70ce376a4bb9dde080c13b6a6bf7d3057d
Author: Hugh Davenport <email address hidden>
Date: Fri May 4 15:06:32 2012 +1200

    Fix problem with '/' in email addresses (bug #994370)

    Replace it with a '-' character

    Change-Id: I92dff41a592748695c59582f69c00f000f555832
    Signed-off-by: Hugh Davenport <email address hidden>

Changed in mahara:
status: In Progress → Fix Committed
Revision history for this message
Hugh Davenport (hugh-davenport) wrote :

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

 status fixreleased
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

iJwEAQECAAYFAlCbHO8ACgkQuMoJ2LQ3zxH8TAP/YN4BiCJZsn5a899/0UzV31Qg
lM8LXAwZWa6zFv6t0BQUHCqe6eFK9wPp51qgCWWXjUZ3vvvVcsyeWp6626aBFKSU
pCQXI9E7huPw802nJQ9WcZXRBUmgw87ww72Tx4mybnu7SPSrkZgXdnPGSMwDs89N
oWvTpl7Xuac48e6p0lU=
=ouU+
-----END PGP SIGNATURE-----

Changed in mahara:
status: Fix Committed → Fix Released
Aaron Wells (u-aaronw)
tags: added: verp
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Related blueprints

Remote bug watches

Bug watches keep track of this bug in other bug trackers.