googleads / google-ads-php

Google Ads API Client Library for PHP
https://developers.google.com/google-ads/api/docs/client-libs/php
Apache License 2.0
292 stars 261 forks source link

No Logging Using REST Transport #414

Open roydidier opened 4 years ago

roydidier commented 4 years ago

I am using Google Ads client library V5 and I have no log generated

I have set logging parameters into google_ads_php.ini [LOGGING] ; Optional logging settings. logFilePath = "/home/metaio-dev-di/google-ads-php/V5/GoogleAdsApi.log" logLevel = "DEBUG"

I don't find what is wrong

thanks

PierrickVoulet commented 4 years ago

Hi @roydidier - I just tested on my side but did not replicate, I see DEBUG logs in the file I specified. Could you give more detail about your setup: OS, PHP, protobuf, transport? Also did you migrate from an older version of the library or is this a brand new setup?

roydidier commented 4 years ago

Hi Pierrick

thany you for you quick answer. Linux version 4.15.0-109-generic (buildd@lgw01-amd64-010) (gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)) PHP 7.2.33 It was working fine with Google Ads API V1 and V2 and I see now that log is not writtten anymore since middle of june . I don't remember having changing anything at this time. I am currently migrating to V5 and get the same issue.

where can I find the protobuf and tranport info?

thanks

roydidier commented 4 years ago

Hello Pierrick

did you get my answer?

Hi Pierrick

thany you for you quick answer. Linux version 4.15.0-109-generic (buildd@lgw01-amd64-010) (gcc version 7.5.0 (Ubuntu 7.5.0-3ubuntu1~18.04)) PHP 7.2.33 It was working fine with Google Ads API V1 and V2 and I see now that log is not writtten anymore since middle of june . I don't remember having changing anything at this time. I am currently migrating to V5 and get the same issue.

where can I find the protobuf and tranport info?

thanks

Didier Roy Head of Software [image: WIHP] http://www.wihphotels.com wihphotels.com http://www.wihphotels.com 77 rue du Faubourg St-Antoine 75011 Paris, France +33 1 53 46 10 63 TEL(ext 47)

Le ven. 4 sept. 2020 à 17:37, Pierrick Voulet notifications@github.com a écrit :

Hi @roydidier https://github.com/roydidier - I just tested on my side but did not replicate, I see DEBUG logs in the file. Could you give more detail about your setup: OS, PHP, protobuf, transport? Also did you migrate from an older version of the library or is this a brand new setup?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/googleads/google-ads-php/issues/414#issuecomment-687224941, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADPKGWSQY4NOFZSCFETHFO3SEECS5ANCNFSM4QYRSDXA .

PierrickVoulet commented 4 years ago

Hello Didier,

I replicate using the REST transport. Could you confirm that you are using REST as a transport on your end? This should be the case if you do not have the gRPC PHP extension installed and configured on the environment.

If you do use REST, the best workaround for now would be to switch to gRPC. You would be able to do this following the instructions in the requirements section of our documentation.

roydidier commented 4 years ago

Hello Pierrick

it was this. it is working fine now

thank you for your help

Didier Roy Head of Software [image: WIHP] http://www.wihphotels.com wihphotels.com http://www.wihphotels.com 77 rue du Faubourg St-Antoine 75011 Paris, France +33 1 53 46 10 63 TEL(ext 47)

Le mar. 8 sept. 2020 à 16:57, Pierrick Voulet notifications@github.com a écrit :

Hello Didier,

I replicate using the REST transport. Could you confirm that you are using REST as a transport https://developers.google.com/google-ads/api/docs/client-libs/php/transport on your end? This should be the case if you do not have the gRPC PHP extension installed and configured on the environment.

If you do use REST, the best workaround for now would be to switch to gRPC. You would be able to do this following the instructions in the requirements section of our documentation https://github.com/googleads/google-ads-php#requirements.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/googleads/google-ads-php/issues/414#issuecomment-688934503, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADPKGWSN6O47EPO25MQ32GTSEZA6FANCNFSM4QYRSDXA .

PierrickVoulet commented 4 years ago

I dug a bit more on this: this is unrelated to the interceptor reimplementation that has been done in V3 as I replicate the issue with V2.

PierrickVoulet commented 3 years ago

This feature depends on the gax-php dependency, a feature request googleapis/gax-php/issues/234 was created on their repository.

jabeztadesse commented 11 months ago

Enabling gRPC solved it for me ... Thanks @PierrickVoulet! 👍