Skip to content
Snippets Groups Projects

remove not necessary content header requirement

Merged Rustam Lotsmanenko (EPAM) requested to merge fix-get-query-endpoint into master

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

Merge request pipeline #170803 passed

Pipeline: Storage

#170804

    Merge request pipeline passed for 34196447

    4 environments impacted.
    Approved by

    Merged by Riabokon Stanislav(EPAM)[GCP]Riabokon Stanislav(EPAM)[GCP] 2 years ago (Mar 8, 2023 11:22am UTC)

    Merge details

    • Changes merged into master with 501681ed (commits were squashed).
    • Deleted the source branch.

    Pipeline #171210 passed with warnings

    Pipeline passed with warnings for 501681ed on master

    4 environments impacted.

    Activity

    Filter activity
    • Approvals
    • Assignees & reviewers
    • Comments (from bots)
    • Comments (from users)
    • Commits & branches
    • Edits
    • Labels
    • Lock status
    • Mentions
    • Merge request status
    • Tracking
    Please register or sign in to reply
    Loading