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

Unable to see error of retrier in OCI SDK #293

Open
kunarkkhewal opened this issue Jun 26, 2024 · 2 comments
Open

Unable to see error of retrier in OCI SDK #293

kunarkkhewal opened this issue Jun 26, 2024 · 2 comments
Labels
SDK Issue pertains to the SDK itself and not specific to any service

Comments

@kunarkkhewal
Copy link

kunarkkhewal commented Jun 26, 2024

In my project, I use OCI object storage, while running on local env. everything is working fine, but on deployment, I am receiving errors like this

Request failed with Exception : [object Object]
Retrying request -> Total Attempts : 1, Retrying after 1.686 seconds...

I am using 'oci-sdk' package for JS/TS
After digging into the package under oci-common/lib/retrier.js I found in line 179
console.warn(`Request failed with Exception : ${lastKnownError}\nRetrying request -> Total Attempts : ${waitContext.attemptCount}, Retrying after ${delayTime} seconds...`);

this should be updated to
console.warn(`Request failed with Exception : ${JSON.stringify(lastKnownError, null, 2)}\nRetrying request -> Total Attempts : ${waitContext.attemptCount}, Retrying after ${delayTime} seconds...`);

because in string literal if we try to print any object it shows as [object Object]

This can act as a blocker in the development because proper error is not shown.

Thank you

@kunarkkhewal kunarkkhewal changed the title Unable to see error of retrier in OCI object storage Unable to see error of retrier in OCI SDK Jun 26, 2024
@jyotisaini jyotisaini added the SDK Issue pertains to the SDK itself and not specific to any service label Aug 5, 2024
@jyotisaini
Copy link
Contributor

Thanks for notifying @kunarkkhewal . We will update the error message to reflect proper error.

@willy-r
Copy link

willy-r commented Oct 2, 2024

Hello! Is this already fixed on current version? (2.95.1)
Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
SDK Issue pertains to the SDK itself and not specific to any service
Projects
None yet
Development

No branches or pull requests

3 participants