This appears after requesting a new password.

Your password and further instructions have been sent to your e-mail address.

warning: Cannot modify header information - headers already sent by (output started at /modules/securesite/securesite.inc:41) in /modules/securesite/securesite.module on line 66.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

batigolix’s picture

Version: master » 4.6.x-1.x-dev

this bug still exists in the 4.6.0 version

the full error is

 warning: Cannot modify header information - headers already sent by 
 (output started at  /var/www/drupal/drupal-4.6.5/modules/securesite/securesite.inc:39) in  
 /var/www/drupal/drupal-4.6.5/modules/securesite/securesite.module on line 66.
NaX’s picture

Status: Active » Needs review

Try removing the securesite_goto(); on line 45 in the securesite.inc file. It is printing out error messages on line 39 and 43, then it is redirecting on line 45. This redirect should be unnecessary since you would want the users to see the message. The exit on line 56 should end page output.

Junyor’s picture

While that does fix the error message, it also leaves me on the logout page. So, if I try to reload the page, I'll end up resubmitting the POST information from the form and my password will change again. If I try to go back to the password form page and reload, I get logged out immediately (since I'm on the logout page). There needs to be a redirect.

Junyor’s picture

FileSize
1.73 KB

I think I've solved this. I left the _goto() in place and added in a session variable. At the top of _init(), if that variable is set, I output the password message and exit(). Seems to work well.

This approach also makes it possible to simply reload the page showing the password message to get an auth dialog. That didn't work before (probably because the _goto() failed).

Junyor’s picture

Here's a patch for HEAD.

Junyor’s picture

Version: 4.6.x-1.x-dev » master
Darren Oh’s picture

Version: master » 4.6.x-1.x-dev
Status: Needs review » Needs work

Committed to DRUPAL-4-7 and HEAD. The 4.6 patch won't commit after the patch for issue 28408 was applied.

Darren Oh’s picture

Committed to DRUPAL-4-6.

Darren Oh’s picture

Status: Needs work » Fixed
Anonymous’s picture

Status: Fixed » Closed (fixed)

  • Commit be587cf on 5.x-1.x, 6.x-1.x, 6.x-2.x, 7.x-2.x, master, 8.x-1.x by Darren Oh:
    #30493 Headers output after password request. By Junyor.