[Bb-trac] Re: [bbPress] #1117: bb-settings.php does not preserve $bb->sitecookiepath

bbPress bb-trac at lists.bbpress.org
Mon Jun 15 04:57:55 GMT 2009


#1117: bb-settings.php does not preserve $bb->sitecookiepath
-------------------------+--------------------------------------------------
 Reporter:  junsuijin    |        Owner:  junsuijin       
     Type:  defect       |       Status:  new             
 Priority:  high         |    Milestone:  1.0             
Component:  integration  |      Version:  1.0-rc-2 (trunk)
 Severity:  major        |   Resolution:                  
 Keywords:  has-patch    |  
-------------------------+--------------------------------------------------
Comment (by junsuijin):

 As I just mentioned on the bbPress.org forum:
 I tested a site with WP in the base folder and bbP in a subfolder of WP,
 and a site with WP in a subfolder of the base folder with bbP in a
 subfolder of WP. Neither site needs anything added to either config file
 (in terms of the suggestions from the admin pages), except that the 3
 secret keys match between them (my nonce keys also match but that
 shouldn't matter). It is also necessary to fill in the wp-admin's bbPress
 Integration Plugin settings, and the bb-admin's WordPress Integration
 settings.

 On any sites I've tested where I have WP in a subfolder of the base
 folder, and bbP in a subfolder of the WP folder, bbP sets 2 logged_in
 cookies, one with a trailing / and one without, which poses a problem
 since WP doesn't get rid of the one without a trailing / and thus WP
 cannot logout from bbP if the user logs in from bbP. After patching bb-
 settings.php with 1117.patch and testing it on both sites mentioned, and
 an additional site with WP in a subfolder of the base folder and bbP in a
 subfolder of WP, I found all sites integrating as expected and bbP would
 only set one logged_in cookie in all cases, as expected.

-- 
Ticket URL: <http://trac.bbpress.org/ticket/1117#comment:2>
bbPress <http://bbpress.org/>
Innovative forum development


More information about the Bb-trac mailing list