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

LANTERN-721: Increased the time delay between CHPL API Requests to 1.2 seconds #378

Merged
merged 2 commits into from
Aug 8, 2024
Merged

LANTERN-721: Increased the time delay between CHPL API Requests to 1.2 seconds #378

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5169341
Increased the time delay between CHPL API Requests to 1.2 seconds
rishi-salunkhe-mettle Jul 30, 2024
586625c
Added a comment explaining the code change
rishi-salunkhe-mettle Aug 1, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion endpointmanager/pkg/chplquerier/chplquerier.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,8 @@ func makeCHPLURL(path string, queryArgs map[string]string, pageSize int, pageNum
func getJSON(ctx context.Context, client *http.Client, chplURL *url.URL, userAgent string) ([]byte, error) {
// request ceritified products list
// Adds a short delay between request
time.Sleep(time.Duration(500 * time.Millisecond))
// LANTERN-721: Increased the delay to 1.2 seconds since the CHPL API rate limit is one request per second
time.Sleep(time.Duration(1200 * time.Millisecond))
req, err := http.NewRequest("GET", chplURL.String(), nil)
if err != nil {
return nil, errors.Wrap(err, "creating http request failed")
Expand Down
Loading