914004 - [email] Bad recipients with clearly invalid e-mail addresses are not detected prior to sending, send does not report specific error codes on failure

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Gaia::E-Mail, enhancement)

Description

[lecky]

User Agent: Mozilla/5.0 (Windows NT 6.1; rv:23.0) Gecko/20100101 Firefox/23.0 (Beta/Release)
Build ID: 20130814063812

Steps to reproduce:

User Agent: Mozilla/4.0 (compatible; MSIE 7.0; Windows NT 6.1; Trident/4.0; SLCC2; .NET CLR 2.0.50727; .NET CLR 3.5.30729; .NET CLR 3.0.30729; Media Center PC 6.0; aff-kingsoft-ciba; .NET4.0C; .NET4.0E)

Steps to reproduce:

【Detail Description*】：
There is no warning(eg. the dress is invalid,pls check it.) when sending a new email with invalid address(eg. xxxxx),
【Repro Steps*】：
1、open email app
2. write a new email with a invalid address(eg.  123456)
3. click send button.





Actual results:

【Real Result*】：
Alert sending email failed.

【Test Count*】：5
【Found Count*】：5
【Gaia commit ID*】:134356dc21d0627455f8bc536b140a18f39e20e9
【Gecko commit ID*】:2e2b4393b0581ef72a1cb70fc5f002cd82a9302a
【Gonk commit ID*】:ca9192d7ef6037016d4723647d86b72f7a410633


Expected results:

【Expect Result*】：
alert the address is invalid, please check it.

Comment 1

[Wayne Chang [:wchang]]

Dominic, I think this would be classified as a feature request?

NI myself to revisit

Comment 2

[lecky]

to Wayne:
Could you tell me whether this request is supported on V1.1HD or not?

Comment 3

[Dominic Kuo [:dkuo]]

Wayne, yes this should be a feature request, I took a quick test and found email didn't check if the address is valid format or not.

Andrew, is this already in the backlog or in the productivity team sprint planning? thanks.

Comment 4

[Wayne Chang [:wchang]]

(In reply to lecky from comment #2)
> to Wayne:
> Could you tell me whether this request is supported on V1.1HD or not?

Comment 3 answers your question.

Not for v1.1/HD.

Comment 5

[Andrew Sutherland [:asuth] (he/him)]

There's no existing bug for this.  There are 2 things we can do here:

- Detect clearly invalid e-mail addresses before attempting to send.  This will benefit asynchronous mail sending too.

- Add more strings to the UI describing the type of failure encountered.

The SMTP back-end will tell us things like 'bad-recipient'/'bad-sender'/'bad-message'/'server-maybe-offline'/etc, but we only ever generate a single string.  Note that bad-recipient could include completely invalid e-mail addresses, or in the event the SMTP server we are using attempts to resolve locally hosted messages, it may be able to indicate that some of the recipients simply don't exist.  (For example, typing '[e-mail]' when you mean '[e-mail]'.)

The ActiveSync back-end does not generate as many error codes, although we could make it smtarter.


I'm adding this to our koi triage list since this is something we should discuss.  Note that the bad case is not as bad as it could be for SMTP.  Since we only generate the message buffer when it comes time to issue the DATA command and we get 'RCPT TO' failures prior to that, we should bail without having constructed the full message buffer or having sent any of the body including potentially large attachments, etc.

Comment 6

[Dylan Oliver [:doliver]]

triage: feature request, moving to consideration for 1.3

Comment 7

[lecky]

This can'e be fixed on V1.1HD, could you accept it?

Comment 8

[Beatriz Rodríguez [:brg]]

(In reply to lecky from comment #7)
> This can'e be fixed on V1.1HD, could you accept it?
Agree with comment 4. We are happy to see that it can be consider as a new feature for v1.3.

Comment 9

[Andrew Sutherland [:asuth] (he/him)]

A UX plan for detecting and allowing editing of erroneous addresses has been detailed in bug 949498 which should address this bug.  Marking us as dependent on that bug.

Comment 11

[BMO Automation]

Firefox OS is not being worked on