search

deltachat / message-parser

Parsing of Links, Email adresses, simple text formatting (markdown subset), user mentions, hashtags and more in DeltaChat messages.
https://deltachat.github.io/message-parser/
Other
12 stars 2 forks source link

quoted email address is parsed too greedy and is wrong by that #78

Open r10s opened 5 months ago

r10s commented 5 months ago

in 1.46.4 (maybe also before), an email address written into quotes becomes tappable including the quotes.

tapping, the email address including the quotes is set as address to the core. the quotes in the alert are part of the address, not to surround the placeholder:

Screenshot 2024-06-15 at 12 00 24

even if " can be a valid characters of the local part (never seen such an address), the top-level-domain de" is definitely invalid.

interestingly, messages can still be sent, despite of the wrong address - either core or some relays are graceful here - maybe accidentally and also buggy? -, but we should not depend on that.

filing the issue here as it is unclear if it is message-parser or sth. else and how to fix. android/ios do not have the issue.

Simon-Laux commented 5 months ago

filing the issue here as it is unclear if it is message-parser or sth. else and how to fix. android/ios do not have the issue.

can only be message-parser

Simon-Laux commented 4 months ago

I have a new case: also happens on "@Name"

Simon-Laux commented 4 months ago

another case is trailing dots like reach me at example@example.com.... or not... your choice..., there the email parsing is too greedy and returns: example@example.com... (last dot is correctly ignored but the extra dots before that not)