297821 - Error messages from the sendmail wrapper may be displayed when it cannot deliver emails

Status: RESOLVED INVALID

(Bugzilla :: Bugzilla-General, defect)

Description

[Amy Lashley]

User-Agent:       Mozilla/4.0 (compatible; MSIE 6.0; Windows NT 5.1; SV1; .NET CLR 1.1.4322)
Build Identifier: Bugzilla 2.18.1

If I request that my password be emailed to me, but then I go back to the top 
of the form and successfully login with my username and password (say I 
suddenly remember the password), it displays the next page, but displays it as 
raw code. If I press back and then login again, the page renders correctly. I'm 
not sure if this has anything to do with my smtp server not being setup 
correctly yet, but it doesn't seem like it should.

Reproducible: Always

Steps to Reproduce:
1.Access login page
2.Enter valid email address, then press "Submit Request"
  System display page advising that password will be mailed.
3.Press "Back"
4.Fill in top form with correct username and password, then press "Login"
  http://localhost/query.cgi is displayed, but renders only with raw code 
displayed
Actual Results:  
http://localhost/query.cgi is displayed, but renders only with raw code 
displayed

Expected Results:  
http://localhost/query.cgi is displayed correctly as a web page (can't see code)

Comment 1

[Max Kanat-Alexander]

Are you actually running this Bugzilla on Windows, or is the OS just set that
way on this bug because you reported it from a Windows machine?

This sounds like a configuration problem somewhere in your web server or your
web browser.

Comment 2

[:glob ✱]

i'm willing to bet this is running on IIS and at the top of the page with raw
html is an error message relating to email delivery problems.

Comment 3

[Amy Lashley]

(In reply to comment #1)

Yes, this is actually on Windows XP. It could very well be a configuration 
problem. I just set it up yesterday. Still haven't got smtp working correctly. 
But it still seems like it should handle the error better if that's the case 
and not just return a page of html code.


> Are you actually running this Bugzilla on Windows, or is the OS just set that
> way on this bug because you reported it from a Windows machine?
> This sounds like a configuration problem somewhere in your web server or your
> web browser.

Comment 4

[Amy Lashley]

(In reply to comment #2)

It's running with apache. the error at the top of the html is related to email 
delivery problems. But should the error be handled better?

sendmail: error during delivery: 5.5.4 Invalid Address

> i'm willing to bet this is running on IIS and at the top of the page with raw
> html is an error message relating to email delivery problems.

Comment 5

[Marc Schumann [:Wurblzap]]

From what I gather, you're being bitten by IIS redirecting STDERR to STDOUT. I
think we want sendmail error messages to appear in the Apache error log (which
is where Apache sends STDERR), so I'm all for WONTFIXing this.

Should we change the summary of the bug?

Comment 6

[:glob ✱]

in comment #4 amy said this is on apache not iis.

the problem is when my sendmail wrapper fails it writes to stdout not stderr. 
this is to ensure that the error is visible on the web page even with apache --
windows admins aren't used to looking in log files for errors.  and even if it
printed to stderr, iis would redirect to stdout.

i think the crux of the problem is we try to send an email before returning the
html -- perhaps that should be swapped?

Comment 7

[Marc Schumann [:Wurblzap]]

Ah, ok, I read "it's running with Apache" as "it's working when I'm using Apache".

I do think the wrapper should write to STDERR. I can't think of an Apache admin
not getting used to looking into the error log -- other programs write there all
the time. Hey, Bugzilla does.

Sending mail later is bug 284184.

Any ideas why the error is displayed on query.cgi instead of on token.cgi?

Comment 8

[Frédéric Buclin]

Someone should update the bug summary. I was going to INVALIDate it when reading it.

Comment 9

[Frédéric Buclin]

byron, I hope this bug summary is closer to reality.

Comment 10

[:glob ✱]

sendmail now writes to stderr, so this bug is either invalid or a duplicate of bug 284184.

(flips a coin)

--> invalid