RuntimeException: Failed to start the session because headers have already been sent

Created on 27 September 2022, almost 2 years ago
Updated 25 September 2023, 9 months ago

Problem/Motivation

Every login raises:

"RuntimeException: Failed to start the session because headers have already been sent by "/var/www/Currents-D9/vendor/symfony/http-foundation/Response.php" at line 368. in Symfony\Component\HttpFoundation\Session\Storage\NativeSessionStorage->start() (line 152 of /var/www/Currents-D9/vendor/symfony/http-foundation/Session/Storage/NativeSessionStorage.php)"

from ...o365/callback.

Despite the error, all logins are all successful.

Steps to reproduce

Login -- redirect /user/login to o365 login is checked in settings.

πŸ› Bug report
Status

Fixed

Version

3.0

Component

Code

Created by

πŸ‡ΊπŸ‡ΈUnited States frymorgan

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • @fabianderijk opened merge request.
  • Status changed to Needs review over 1 year ago
  • πŸ‡³πŸ‡±Netherlands fabianderijk Alphen aan den Rijn

    Thanks for the help and the pointers. I've just created a MR which includes the code from @phma. You can review this.

    @phma, regarding the help. You are always welcome in helping this module grow both in usage as in maturity. We are planning to add some test coverage to this module, if you are inclined to help, this would be our main focus for the foreseeable future.

  • Assigned to mikechan
  • πŸ‡³πŸ‡±Netherlands fabianderijk Alphen aan den Rijn
  • Status changed to RTBC 9 months ago
  • πŸ‡¨πŸ‡­Switzerland phma Basel, CH

    @fabianderijk Thanks for creating the MR. I never go around doing it myself. Also, our website no longer needs the destination parameter to work so desperately. Still, I've applied the patch on our site and can confirm that it is working locally on my machine. At least I can help others, and it will be working if our site ever requires it.

    As I'm changing roles soon, my focus is going to shift, and I'm probably no longer able to help you guys, sorry. But I hope I can continue to contribute in some ways in the future. In the meantime, keep up the good work!

    I'm marking this as RTBC, but feel free to change it back if you want someone else to test it.

  • Status changed to Fixed 9 months ago
  • πŸ‡³πŸ‡±Netherlands fabianderijk Alphen aan den Rijn

    This has now been added to the 3.0.x branch dev release, and is added to the new 5.0.0 release tag

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

Production build 0.69.0 2024