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.

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

[Backport 1.x] Added support for headers for ExtensionRestRequest #662

Merged
merged 1 commit into from
Apr 8, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link

Backport dfdb050 from #643.

* added uri and httpVersion to HttpRequest

Signed-off-by: Daulet <[email protected]>

* added SDKHttpRequest, SDKRestRequest

Signed-off-by: Daulet <[email protected]>

* spotless applied

Signed-off-by: Daulet <[email protected]>

* moved SDKHttpRequest and SDKRestRequest to org.opensearch.sdk.rest

Signed-off-by: Daulet <[email protected]>

* tests changed for new ExtensionRestRequest

Signed-off-by: Daulet <[email protected]>

* TestSDKRestRequest created

Signed-off-by: Daulet <[email protected]>

* corrected test for xContentType, added test for contentParser()

Signed-off-by: Daulet <[email protected]>

* replaced .map() with mapStrings() in contentParser()

Signed-off-by: Daulet <[email protected]>

* Update SDKHttpRequest.java

Signed-off-by: Daulet Nassipkali <[email protected]>

* Fix tests for new format

Signed-off-by: Daniel Widdis <[email protected]>

* Use correct Test annotation so tests actually run

Signed-off-by: Daniel Widdis <[email protected]>

---------

Signed-off-by: Daulet <[email protected]>
Signed-off-by: Daulet Nassipkali <[email protected]>
Signed-off-by: Daniel Widdis <[email protected]>
Co-authored-by: Daniel Widdis <[email protected]>
(cherry picked from commit dfdb050)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@owaiskazi19
Copy link
Member

And the workflow started working again!

@dbwiddis
Copy link
Member

dbwiddis commented Apr 7, 2023

Gradle check failing until 2.x backport of companion PR is merged: opensearch-project/OpenSearch#7055

@dbwiddis dbwiddis merged commit a18fa9f into 1.x Apr 8, 2023
@dbwiddis dbwiddis deleted the backport/backport-643-to-1.x branch April 8, 2023 00:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants