search

gigya / drupal8

Gigya Drupal 9/10 GConnector
GNU General Public License v2.0
4 stars 15 forks source link

Improve error logging #106

Closed iz7crx closed 1 year ago

iz7crx commented 2 years ago

validateAndFetchRaasUser could fail due to lack of permissions on Gigya side. It could save time to explicitly suggest a permission check.

Ynhockey commented 2 years ago

Hi, is the current error with the call ID and error details not detailed enough? Just want to understand because I'd rather not have error-specific logs, but we can expose more details in the log per PHP SDK.

iz7crx commented 2 years ago

Hi,

we got "Gigya API call error: Permission denied, Call ID: X" and then "Invalid user. Guid: X" on the log.

Gigya console manager told us it was an integration/configuration issue on drupal side.

We did extended debugging and got 403007.

Additionaly It has been hard to find error code descriptions as it happens web integrators do not own Gigya and cannot access to Gigya docs. Altough 403 is self-explaining for me, it isn't so for everyone.

We found 403007 description on some cache results somewhere and it allowed us to request a configuration check on Gigya side and finally he fixed permissions.

I suggest this error should be logged because allows non-Gigya people to address troubleshooting in "silos" scenarios.

Ciao,

Dario

levistepanov commented 2 years ago

Hi iz7crx, Gigya docs are publicly available, all errorCodes can be found here: https://help.sap.com/viewer/8b8d6fffe113457094a17701f63e3d6a/GIGYA/en-US/416d41b170b21014bbc5a10ce4041860.html

iz7crx commented 2 years ago

Hi iz7crx, Gigya docs are publicly available, all errorCodes can be found here: https://help.sap.com/viewer/8b8d6fffe113457094a17701f63e3d6a/GIGYA/en-US/416d41b170b21014bbc5a10ce4041860.html

Thank you for your reply, 403007 code was found only through extended debugging, the only messages on drupal log were

Gigya API call error: Permission denied Invalid user

Ynhockey commented 1 year ago

I apologize for the long wait, but I have added error code to the log. There are too many possible error codes to individually handle each one, but having the code in the log will help anyone troubleshoot these issues. Moreover: it is possible to get a lot of information about any error through the callID, which is also written to the log.