Skip to content

Update request.py to add more "ok" response. For performance purpose.

Bruce Jin requested to merge extend-ok-response into master

Type of change

  • Bug Fix
  • Feature

Does this introduce a change in the core logic?

  • [No]

Does this introduce a change in the cloud provider implementation, if so which cloud?

  • AWS
  • Azure
  • GCP
  • IBM

Updates description?

Describe your code changes in details for reviewers (links on Gitlab issues, etc.)

Currently when running manifest ingestion by reference, one single record will trigger more than 1 PUT call to the storage service. This is due to this API will return a 201 CREATED if it works, which is not an OK response within file common-python-sdk/osdu_api/utils/request.py. We need to include more acceptable status codes to avoid time wasting.

Merge request reports