Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workers crashing on 1.5.2 #674

Closed
damiantw opened this issue Apr 4, 2023 · 2 comments
Closed

Workers crashing on 1.5.2 #674

damiantw opened this issue Apr 4, 2023 · 2 comments

Comments

@damiantw
Copy link

damiantw commented Apr 4, 2023

  • Octane Version: 1.5.2
  • Laravel Version: 10.6.0
  • PHP Version: 8.2.4
  • Server & Version: RoadRunner 2.12.3 (Also, Vapor deployment w/ Octane enabled)
  • Database Driver & Version: Postgres 15.2

Description:

The latest Octane release seems to have a regression related to 7fecb1d.

Workers crash after serving an initial request. Workers stop crashing when I take any of the following actions.

  1. Downgrade to Octane 1.5.1
  2. Remove 'session' and 'session.store' from the octane.warm configuration
  3. Remove \Laravel\Octane\Listeners\GiveNewApplicationInstanceToSessionManager::class as a RequestReceived listener

I'm having trouble finding any useful error output from my local environment (nothing is written to the Laravel log). However, after deploying 1.5.2 to the Vapor environment our logs filled w/ error messages like the following.

{
  "exception": {
    "class": "Illuminate\\Contracts\\Container\\BindingResolutionException",
    "message": "Target class [config] does not exist.",
    "code": 0,
    "file": "/var/task/vendor/laravel/framework/src/Illuminate/Container/Container.php:916",
    "previous": {
      "class": "ReflectionException",
      "message": "Class \"config\" does not exist",
      "code": -1,
      "file": "/var/task/vendor/laravel/framework/src/Illuminate/Container/Container.php:914"
    }
  }
}

Steps To Reproduce:

Octane 1.5.2

Screenshot 2023-04-04 at 2 13 18 PM

Octane 1.5.1

Screenshot 2023-04-04 at 2 15 44 PM

This was referenced Apr 4, 2023
@nunomaduro
Copy link
Member

Fixed on 1.5.3.

@taylorotwell
Copy link
Member

Also need Laravel framework 10.6.1 FYI.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants