265300 - Check for subject in email form submission should be case-insensitive

Status: RESOLVED FIXED

(Core :: DOM: Core & HTML, defect)

Description

[Thomas R. Scott]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7.3) Gecko/20041001 Firefox/0.10.1
Build Identifier: Mozilla/5.0 (Windows; U; Windows NT 5.0; rv:1.7.3) Gecko/20041001 Firefox/0.10.1

In IE, one can have the form send e-mail with a specific subject line such as
"[ISUS Web] Dealer Inquiry".  However, with Firefox, the subject is replaced
with "Form Post From Mozilla".

Sample code used to designate e-mail address and subject are:

<form method="POST" enctype="text/plain"
action="mailto:[e-mail];
[e-mail]?Subject=[ISUS Web] Dealer Inquiry">

form body . . .

</form>

Reproducible: Always
Steps to Reproduce:
1. Noted in detail and in sample web page referenced above.
2.
3.

Actual Results:  
Noted in detail above.


Expected Results:  
Perform as IE does, in this instance.

For severity, I selected Minor, however, there does not seem to be an easy work
around.

Thanks

[e-mail]

Comment 1

[franz.ose]

Also, if there is no subject specified it should say
"Form Post From Firefox" (or something even better like "E-Mail send from
www.something.com")
and not "Form Post From Mozilla"

Comment 2

[Robert Kaiser]

The general part of that might be bug 207020 - I filed bug 287695 about the
issue that we have a hardcoded app name there (which soon won't even exist any
more on trunk).

Comment 3

[Thomas R. Scott]

Is there any chance of this being fixed soon?

I originally posted this as a Minor bug.  However, with Mozilla becoming more
popular it is a growing problem.  Given how long this has taken, I changed the
severity to Normal.

 - Tom

Comment 4

[Simon Paquet [:sipaq]]

Should be fixed with the checkin for bug 287695. Please reopen if this is not
the case.

Comment 5

[Thomas R. Scott]

As one can see, I posted this bug over two years ago.  I recently received an e-mail indicating that the bug report status may have changed.  Indeed the report was at a status of "Resolved Fixed".

I just tested it with Firefox 2 (standard release) on a Win 2K platform.  The bug has NOT been resolved or fixed.

As such, I'm reopening the bug report.

 - Tom

Comment 6

[Phil Ringnalda (:philor)]

We're checking for only a lower-case "subject", not "Subject" - I'd be surprised if anything else makes that mistake, so you probably can work around it until Firefox 3 ships by using ?subject= instead.

Comment 7

[Phil Ringnalda (:philor)]

Attachment: testcase

Comment 8

[Phil Ringnalda (:philor)]

Attachment: Fix v.1

I guess this is the right place to break the line.

Comment 9

[Phil Ringnalda (:philor)]

(In reply to comment #6)
> by using ?subject= instead.

Hmm, actually, at least Thunderbird and Evolution don't seem to accept a lowercase subject, only Subject. I'm not quite sure whether that's their problem, and we should be passing through subject=foo anyway, or we should be doing a case-sensitive check for Subject.

Comment 10

[Boris Zbarsky [:bzbarsky]]

Comment on attachment 256632 [details] [diff] [review]
Fix v.1

Doing this case-insensitive is correct -- this is supposed to be an RFC822 header name, and those are case-insensitive.

As for Thunderbird, that's odd.  I see it doing case-insensitive compares in the relevant code...

Comment 11

[Phil Ringnalda (:philor)]

content/html/content/src/nsFormSubmission.cpp 1.53