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

Composer package br33f/php-ga4-mp v0.1.5 not compatible #7113

Closed
Christophe22PE opened this issue May 15, 2024 · 2 comments
Closed

Composer package br33f/php-ga4-mp v0.1.5 not compatible #7113

Christophe22PE opened this issue May 15, 2024 · 2 comments
Labels
needs investigation An issue that needs further investigation by a developer to determine the root cause.

Comments

@Christophe22PE
Copy link

Installation Method

Docker Installation

AzuraCast Release Channel

Stable Channel

Current AzuraCast Version

0.19.7 Stable • Docker • PHP 8.3

What happened?

Hi,

After an Azuracast update, composer packages has been updated too.

We noticed that the br33f/php-ga4-mp composer package in its version 0.1.5 is not compatible with Azuracast.

We downgrade to version 0.1.4, and it works again !

Best regards.

Relevant log output

[2024-05-15T19:45:39.642301+00:00] AzuraCast.INFO: Dispatching test web hook "Googleanalytics4"... [] []
[2024-05-15T19:45:39.644967+00:00] AzuraCast.DEBUG: Writing local nowplaying text file... [] []
[2024-05-15T19:45:39.645683+00:00] AzuraCast.DEBUG: Writing static nowplaying text file... [] []
[2024-05-15T19:45:39.647571+00:00] AzuraCast.DEBUG: HTTP client POST call to http://localhost:6025/api produced response 200 [] []
[2024-05-15T19:45:39.649445+00:00] AzuraCast.ERROR: /var/azuracast/www/vendor/br33f/php-ga4-mp/src/Dto/Request/BaseRequest.php L368: Parameter "client_id" is required. {"exception":"[object] (Br33f\\Ga4\\MeasurementProtocol\\Exception\\ValidationException(code: 1000002): Parameter \"client_id\" is required. at /var/azuracast/www/vendor/br33f/php-ga4-mp/src/Dto/Request/BaseRequest.php:368)"} []
@Christophe22PE Christophe22PE added the needs investigation An issue that needs further investigation by a developer to determine the root cause. label May 15, 2024
BusterNeece added a commit that referenced this issue May 16, 2024
@BusterNeece
Copy link
Member

@Christophe22PE The latest push to the Rolling Release branch should fix this, but if you can, I'd recommend updating to the latest Rolling Release and testing it out to confirm.

@Christophe22PE
Copy link
Author

Hello, Thank you for your feedback, I was on the weekend, it works now, thank you.

Best Regards.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs investigation An issue that needs further investigation by a developer to determine the root cause.
Projects
None yet
Development

No branches or pull requests

2 participants