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

fix(cb2-13831): retry on status of 0 #1622

Merged
merged 2 commits into from
Oct 14, 2024
Merged

fix(cb2-13831): retry on status of 0 #1622

merged 2 commits into from
Oct 14, 2024

Conversation

pbardy2000
Copy link
Contributor

VTM Sentry - Http failure response for /v1/document-retrieval: 0 Unknown Error

Have checked common causes for this occurring (see https://javascript-code.dev/articles/263765532), and can confirm this is neither a misconfigured http client or an invalid URL. It is likely a network/firewall issue or a CORs issue which cannot be fixed client side, so I'm following the recommendation to simply handle and retry the request when a 0 status code is received.

CB2-13831

Checklist

  • Branch is rebased against the latest develop/common
  • Code and UI has been tested manually after the additional changes
  • PR title includes the JIRA ticket number
  • Squashed commits contain the JIRA ticket number
  • Delete branch after merge

@github-actions github-actions bot added the bug Something isn't working label Oct 11, 2024
@pbardy2000 pbardy2000 marked this pull request as draft October 11, 2024 11:55
@pbardy2000 pbardy2000 marked this pull request as ready for review October 11, 2024 12:08
@pbardy2000 pbardy2000 merged commit cfa0937 into develop Oct 14, 2024
11 checks passed
@pbardy2000 pbardy2000 deleted the fix/CB2-13831 branch October 14, 2024 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants