Opened 3 years ago

Closed 3 years ago

#1486808 closed Bugs (fixed)

local part may contain equal sign

Reported by: pommi Owned by:
Priority: 5 Milestone: 0.4.1
Component: Core functionality Version: 0.4-beta
Severity: normal Keywords:
Cc:

Description

I'm using ezmlm-idx mailing list software. When confirming a subscription you have to send an email to an address in this format:

listname-sc.1234567890.key-mylocalpart=mydomain.tld@listdomain.tld

Sending this e-mail fails with the error:

Invalid e-mail address: <email>

The problem is the equal sign (=). According to Wikipedia the equal sign is a valid character in the local-part.

Attached is a patch to resolve this issue when sending an e-mail to an address with an equal sign in the local-part. But I found a couple of different regex-es for detecting/validating e-mail addresses, so I think this patch is not enough to fix the problem completely.

Attachments (2)

rcpt-localpart-equalsign.patch (627 bytes) - added by pommi 3 years ago.
mailto_pattern_rfc5322.patch (669 bytes) - added by pommi 3 years ago.

Download all attachments as: .zip

Change History (6)

Changed 3 years ago by pommi

comment:1 Changed 3 years ago by alec

  • Milestone set to 0.4-stable
  • Resolution set to fixed
  • Status changed from new to closed

Fixed in [d7a5dfa2].

comment:2 in reply to: ↑ description Changed 3 years ago by pommi

  • Resolution fixed deleted
  • Status changed from closed to reopened

Replying to pommi:

But I found a couple of different regex-es for detecting/validating e-mail addresses, so I think this patch is not enough to fix the problem completely.

Attached is a patch for program/include/rcube_string_replacer.php to detect the local part like in program/include/main.inc around line 1591.

Changed 3 years ago by pommi

comment:3 Changed 3 years ago by alec

  • Milestone changed from 0.4-stable to 0.4.1

comment:4 Changed 3 years ago by alec

  • Resolution set to fixed
  • Status changed from reopened to closed

Fixed in [27a96a40]. When we implement IDNA, we'll need to change the regexp to support emails with unicode characters.

Note: See TracTickets for help on using tickets.