Closed (fixed)
Project:
log_stdout
Version:
8.x-2.x-dev
Component:
Code
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Feb 2023 at 08:45 UTC
Updated:
3 Jun 2025 at 10:14 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #2
maico de jongModule works fine on Drupal 10 with a small change mentioned in #3272447: Update to PSR/log v3
Comment #3
hardikpandya commentedI was not able to apply the patch in #2. Got this error:
Comment #6
makazimtingwa commentedCreated the issue fork to install this project in Drupal 10 with:
composer require drupal/log_stdout:dev-3339853-drupal-10-compatibilityAt present, this will install patch #2. You may need to include patch #3 locally.
Comment #7
arturopanettaI tested patch #2 and it works, I can redirect logs to stdout container. Patch #3 which I was unable to apply as it tries to remove some lines of code related to module versioning.
I used to:
- Drupal 10.0.9
- log_stdout branch 1.x
- patches #2
Comment #8
gigimaorThis one is working for me after upgrading to Drupal 10.
Comment #9
tolstoydotcomThe patch in #8 looks like it works. I also ran
phpcs --standard=Drupaland didn't see any warnings.Comment #10
jonmcl commentedPatch in #8 works with with the 8.x-1.3 branch, even though this issue is tagged with the 8.x-2.x-dev branch.
Update: Scratch that. Patch #8 only works with the 1.x-dev version (composer require 'drupal/log_stdout:1.x-dev@dev')
Comment #11
tolstoydotcom8.x-2.x-dev (Last updated: 2 Sep 2020) is marked as only working with D8
8.x-1.x-dev (Last updated: 4 Oct 2021) is marked as working with ^8.8 || ^9 and the patch adds D10 compatibility. That's what I tested.
The main difference between the two dev versions seems to be that 2.x-dev uses a slightly different output format and it also adds user info to the output.
Comment #12
sudishth commentedreroll #8
Comment #13
sudishth commentedpatch for 8.x-1.3
Comment #14
scarer commentedYes #13 can confirm it works.
Comment #17
bladeduMerged and created release 1.4 which is D10 compatible.
Thank you!
Comment #18
rm-rf commentedI'm afraid you will still need to update the composer.json in order for this module to install correctly via Composer on a D10 install.
Comment #19
bladeduYou are right. I didn't check the patch properly, I'm afraid.
Comment #22
bladeduit should be ok now in 1.4.1. sorry about that.
Comment #23
bladeduComment #24
rm-rf commentedThat looks good and the module now installs correctly via Composer on Drupal 10.
Comment #25
fraserthompsonAlso confirming that the dev branch works with Composer on Drupal 10 but the current stable release doesn't and throws an error.
Comment #26
bladeduI apologize for the inconvenience. I messed up with tag 1.4 and I couldn't amend it.
I've just releases 1.5 which can be pulled with latest Drupal 10 release.
Comment #28
pbonnefoi commentedComment #29
pbonnefoi commentedComment #30
pbonnefoi commented