BugsFacebook::__construct() does session_start() whether it makes sense or not
- Status: Closed
- Priority: None
- Updated: December 15, 2012 at 10:05am
Latest response from Facebook
Eric Osgood
Thanks for the report. We will follow up.
Status changed to Triaged · Priority changed to Low · November 15, 2012 at 9:07am
Facebook::__construct() blindly and stupidly does session_start() whether you need it or not, but there are tons of cases where you don't need to initiate a session at all and so, this will produce PHP errors if output has already been sent, when actually there's no problem at all because you don't need a session in the first place.
| Steps to Reproduce: | bla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla blabla bla bla bla |
|---|---|
| Expected Behavior: | only start session if really needed |
| Actual Behavior: | starts session in the very Facebook constructor. Half of the times you don't need it at all |
| API/SDK: | PHP SDK |
| App IDs: |
|
1 person can reproduce this issue by following these steps
Repro