Skip to content

Conversation

@cx-sarah-chen
Copy link
Contributor

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

References

Implemented retry logic and use both Scans and Project APIs within the Scans service to address 502 errors.

Testing

Checklist

  • I have added documentation for new/changed functionality in this PR (if applicable).
  • I have updated the CLI help for new/changed functionality in this PR (if applicable).
  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@github-actions
Copy link

github-actions bot commented Jan 30, 2025

Logo
Checkmarx One – Scan Summary & Details7f367e7c-eb58-4dfe-ab56-ffaed7fbff1f

Great job, no security vulnerabilities found in this Pull Request

cx-ben-alvo
cx-ben-alvo previously approved these changes Feb 2, 2025
@cx-sarah-chen cx-sarah-chen merged commit d4ef245 into main Feb 2, 2025
10 checks passed
@cx-sarah-chen cx-sarah-chen deleted the feature/saraChen/AddRetryToHttpCall branch February 2, 2025 14:28
cx-anjali-deore pushed a commit that referenced this pull request Jul 3, 2025
…tpCall

Add retry mechanism to 502 response (AST-82566)
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.

4 participants