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

chore: add tracing for tracking disp retries #562

Merged

Conversation

aarlaud
Copy link
Contributor

@aarlaud aarlaud commented Jun 8, 2023

What does this PR do?

Lowers timeout value for dispatcher notifications and log retries failure more clearly.

Any background context you want to provide?

Some very sporadic network communication errors appeared in logs. Attempting to log these more clearly to investigate further.

@aarlaud aarlaud requested a review from a team as a code owner June 8, 2023 08:42
@aarlaud aarlaud force-pushed the chore/lower-timeout-and-retry-dispatcher-advertising branch from 03b0a36 to 682ccee Compare June 8, 2023 09:20
lib/axios.ts Outdated Show resolved Hide resolved
@aarlaud aarlaud force-pushed the chore/lower-timeout-and-retry-dispatcher-advertising branch from 682ccee to dfa21a0 Compare June 9, 2023 08:41
@aarlaud aarlaud merged commit d4dbc91 into master Jun 13, 2023
@aarlaud aarlaud deleted the chore/lower-timeout-and-retry-dispatcher-advertising branch June 13, 2023 09:52
@snyksec
Copy link

snyksec commented Jun 22, 2023

🎉 This PR is included in version 4.157.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants