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

No logs streamed #46

Closed
Norgul opened this issue Sep 1, 2022 · 8 comments
Closed

No logs streamed #46

Norgul opened this issue Sep 1, 2022 · 8 comments

Comments

@Norgul
Copy link
Contributor

Norgul commented Sep 1, 2022

Hello,

I tried using the package and it loads log files just fine (I can even download them with full content), however I am not getting anything in the view.

I am using non-published conf (all defaults), latest version (1.2.4) with L9.13.0.
Also tried chmod -R 777 storage, didn't help. Also tried flushing cache.

I am running a Docker container with Roadrunner server.

image

@arukompas
Copy link
Contributor

Hey @Norgul , I'm having trouble reproducing this.

I've set up a brand new Laravel project (v9.13.0 like yours), set up Roadrunner, and it works perfectly fine (both in Sail (docker) and native MacOS)

Screenshot 2022-09-01 at 19 41 48

I do have a few questions that may help me figure out the issue.

  1. Are you using a specific Livewire version yourself?

  2. Have you modified the default 'web' middleware group from Laravel? Make sure that cookies and send and a session is started, which is required for Livewire to function properly.

  3. Does this happen to all the logs, or just this one file? Does it have a non-standard log format perhaps? Maybe you can paste the log file contents here.

  4. Lastly, is there any other package or Octane/Roadrunner configuration that's non-standard and you believe you could interfere with this?

Thanks!

@Norgul
Copy link
Contributor Author

Norgul commented Sep 1, 2022

Hey @arukompas, thanks for

  1. We are using Laravel for API only, and it started back in the 7.x version so it has no Livewire as a dependency.
  2. web middleware is intact
  3. It happens to all the logs, and now that you mention it, yes, log format is slightly different:

[2022-08-30 06:16:39.598814] [Process manager] 127.0.0.1 [] testing.INFO: Process type

  1. That's a negative

Third point might be the real culprit here. Is there a way to configure that?

@arukompas
Copy link
Contributor

arukompas commented Sep 1, 2022

@Norgul that's it. It's the custom log format.

Here's an example of the Regex we use to determine when it's the start of a new log entry - https://regexr.com/6t56r

You can see that your example does not get picked up.

Unfortunately, at the moment there's no way to configure the Regex used, BUT I do have plans to introduce "log reader drivers" which can be writte for pretty much any type of log in the future (laravel, mysql, apache, redis, etc).

So for now, if you really want to use this log viewer then I suggest forking the project for your own use and adjusting the Regex expressions found here:

Closing this for now as it's not something I can fix right now, but let me know if there's anything else I can help with.

@arukompas arukompas closed this as not planned Won't fix, can't repro, duplicate, stale Sep 1, 2022
@arukompas
Copy link
Contributor

@Norgul I may have figured something out... 🤔

@arukompas arukompas reopened this Sep 1, 2022
@Norgul
Copy link
Contributor Author

Norgul commented Sep 1, 2022

What about something in the likes of this: #51 ?
Didn't test it, but I can play around if you think it's worth a shot.

@arukompas
Copy link
Contributor

hey @Norgul !

I've just tagged a new release, v1.2.5 which has a slightly different Regex that's more forgiving to custom log formats. Although your custom data (like [Process manager] 127.0.0.1 []) is not in their own columns, it's still clearly visible and accessible!

Screenshot 2022-09-01 at 20 33 56

Hope this helps 😄

As for your PR, I'll review this tomorrow

@arukompas
Copy link
Contributor

marking as fixed, let me know if there's still any issues.

@Norgul
Copy link
Contributor Author

Norgul commented Sep 2, 2022

Yup, it works, thanks!

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

2 participants