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.

Sign up for GitHub

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

Jump to bottom

Improve Retry Logic to Only Retry on Server-Side HTTP Errors #1390

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Improve Retry Logic to Only Retry on Server-Side HTTP Errors #1390

wants to merge 11 commits into from

Commits on Nov 9, 2024

  1. Adding error handling for status code 5xx

    @VishalGawade1
    VishalGawade1 committed Nov 9, 2024
    Configuration menu
    Copy the full SHA
    d8d00bc View commit details
    Browse the repository at this point in the history

  2. resolving merge conflict in go.mod

    @VishalGawade1
    VishalGawade1 committed Nov 9, 2024
    Configuration menu
    Copy the full SHA
    55f448a View commit details
    Browse the repository at this point in the history

  3. updating osv.go comment for handling 200 inside makeRetryRequest

    @VishalGawade1
    VishalGawade1 committed Nov 9, 2024
    Configuration menu
    Copy the full SHA
    c91540b View commit details
    Browse the repository at this point in the history

Commits on Nov 10, 2024

  1. refactor: use standard library for test assertions

    @VishalGawade1
    VishalGawade1 committed Nov 10, 2024
    Configuration menu
    Copy the full SHA
    03db161 View commit details
    Browse the repository at this point in the history

  2. fix: correct makeRetryRequest retry logic and update tests

    @VishalGawade1
    VishalGawade1 committed Nov 10, 2024
    Configuration menu
    Copy the full SHA
    a30e28e View commit details
    Browse the repository at this point in the history

  3. Removing the QueryEndpoint Override from test

    @VishalGawade1
    VishalGawade1 committed Nov 10, 2024
    Configuration menu
    Copy the full SHA
    e297913 View commit details
    Browse the repository at this point in the history

Commits on Nov 11, 2024

  1. Added error handling in makeRetryRequest, removed redundant functions… 

    …, and cleared all lint warnings
    @VishalGawade1
    VishalGawade1 committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    8c369d6 View commit details
    Browse the repository at this point in the history

  2. removed unused dependency in go.mod, reverted constants to original f… 

    …ormat, and restored jitter implementation to avoid code duplication in makeRetryRequest function
    @VishalGawade1
    VishalGawade1 committed Nov 11, 2024
    Configuration menu
    Copy the full SHA
    e81d59d View commit details
    Browse the repository at this point in the history

  3. Merge branch 'main' into main

    @VishalGawade1
    VishalGawade1 authored Nov 11, 2024
    Configuration menu
    Copy the full SHA
    f8676b7 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2024

  1. Merge branch 'main' into main

    @VishalGawade1
    VishalGawade1 authored Nov 14, 2024
    Configuration menu
    Copy the full SHA
    4fa1286 View commit details
    Browse the repository at this point in the history

Commits on Nov 15, 2024

  1. Adding error handling for status code 429

    @VishalGawade1
    VishalGawade1 committed Nov 15, 2024
    Configuration menu
    Copy the full SHA
    a1a3e93 View commit details
    Browse the repository at this point in the history