After upgrade to 7.2, watchdog log got swamped with these messages:

Warning: session_id(): Cannot change session id when headers already sent in drupal_session_initialize() (line 266 of includes/session.inc).
Warning: session_set_save_handler(): Cannot change save handler when headers already sent in drupal_session_initialize() (line 242 of includes/session.inc).

There is a related issue for drupal core, for the statistics module.
https://www.drupal.org/project/drupal/issues/3009351

Based on this patch, the solution appear to be to take out this line:
visitors.exit.inc #164
drupal_bootstrap(DRUPAL_BOOTSTRAP_SESSION);

I did this, and now everything seems to run normally again, also I am not seeing any errors as far as behavior for visitors.

I set this to major, because 1) the core patch was also major and 2) it really swamped the logs, especially on a busy site.

CommentFileSizeAuthor
#3 3053329-3.patch353 bytesjoelpittet

Issue fork visitors-3053329

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

hoporr created an issue. See original summary.

avpaderno’s picture

Issue tags: -undefined
joelpittet’s picture

Version: 7.x-1.5 » 7.x-1.9
Status: Active » Needs review
StatusFileSize
new353 bytes

In patch form, I'm seeing this too.

bluegeek9 made their first commit to this issue’s fork.

  • bluegeek9 committed 1aa2269 on 7.x-1.x
    Issue #3053329: [PHP ≥ 7.2] "session_id(): Cannot change session id"
    
bluegeek9’s picture

Status: Needs review » Fixed

  • bluegeek9 committed d4f4e6c on 7.x-2.x authored by joelpittet
    Issue #3053329 by joelpittet: [PHP ≥ 7.2] "session_id(): Cannot change...
joelpittet’s picture

Thank you @bluegeek9, nice to see this committed!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.