remove not necessary content header requirement
Description:
GET query/records endpoint requires Content-Type
header in the request.
curl --location 'https://community.gcp.gnrg-osdu.projects.epam.com/api/storage/v2/query/records?kind=osdu%3Awks%3Amaster-data--Well%3A1.0.0' \
--header 'data-partition-id: osdu' \
--header 'Content-Type: application/json' \
--header 'Authorization: Bearer
If a header is not provided Storage responds with a 415 Unsupported Media Type
code.
We need to remove this requirement since this endpoint doesn't accept request body, and shouldn't specify content-type
.
How to test:
GET query/records works without Content-Type
header.
curl --location 'https://community.gcp.gnrg-osdu.projects.epam.com/api/storage/v2/query/records?kind=osdu%3Awks%3Amaster-data--Well%3A1.0.0' \
--header 'data-partition-id: osdu' \
--header 'Authorization: Bearer
Changes include:
-
Bugfix (a non-breaking change that solves an issue).
Changes in:
-
Common code
Edited by Rustam Lotsmanenko (EPAM)
Merge request reports
Activity
changed milestone to %M17 - Release 0.20
added Common Code label
requested review from @Stanislav_Riabokon
assigned to @Rustam_Lotsmanenko
added 3 commits
-
ea437e8b...91f0f5ff - 2 commits from branch
master
- 197b8b7a - Merge branch 'master' into fix-get-query-endpoint
-
ea437e8b...91f0f5ff - 2 commits from branch
added 5 commits
-
197b8b7a...94272cb7 - 4 commits from branch
master
- 34196447 - Merge branch 'master' into fix-get-query-endpoint
-
197b8b7a...94272cb7 - 4 commits from branch
@nthakur Could you help review too ?
added MREnhancement label
mentioned in commit 501681ed
Please register or sign in to reply