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] [2.x] Modify to not use URLEncodedUtils (#999) #1012

Merged

Conversation

reta
Copy link
Collaborator

@reta reta commented Jun 5, 2024

Backport of #999 to 2.x

@reta reta force-pushed the modify-to-not-use-url-encoded-utils2.x branch from 1141a3e to 31fcd87 Compare June 5, 2024 18:01
* modify to not use URLEncodedUtils

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* add a license header

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* add pathEncoder test

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* modify to use ByteBuffer in encodeBytes method

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* modify to use URLEncoder

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* delete PathEncoder

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* modify to use URLEncodedUtils per version per Classpath

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* fix to use MethodHandle instead of reflection

Signed-off-by: AntCode97 <dbswns97@gmail.com>

* add CHANGELOG.md

Signed-off-by: AntCode97 <dbswns97@gmail.com>

---------

Signed-off-by: AntCode97 <dbswns97@gmail.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
@reta reta force-pushed the modify-to-not-use-url-encoded-utils2.x branch from 31fcd87 to 4647161 Compare June 5, 2024 18:04
@reta
Copy link
Collaborator Author

reta commented Jun 5, 2024

Seems like backport job didn't trigger :(

@dblock dblock merged commit 8808100 into opensearch-project:2.x Jun 5, 2024
47 checks passed
reta added a commit to reta/opensearch-java that referenced this pull request Jun 5, 2024
…h-project#1012)

* modify to not use URLEncodedUtils



* add a license header



* add pathEncoder test



* modify to use ByteBuffer in encodeBytes method



* modify to use URLEncoder



* delete PathEncoder



* modify to use URLEncodedUtils per version per Classpath



* fix to use MethodHandle instead of reflection



* add CHANGELOG.md



---------

Signed-off-by: AntCode97 <dbswns97@gmail.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Co-authored-by: AntCode97 <50140266+AntCode97@users.noreply.github.com>
dblock pushed a commit that referenced this pull request Jun 5, 2024
* modify to not use URLEncodedUtils



* add a license header



* add pathEncoder test



* modify to use ByteBuffer in encodeBytes method



* modify to use URLEncoder



* delete PathEncoder



* modify to use URLEncodedUtils per version per Classpath



* fix to use MethodHandle instead of reflection



* add CHANGELOG.md



---------

Signed-off-by: AntCode97 <dbswns97@gmail.com>
Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
Co-authored-by: AntCode97 <50140266+AntCode97@users.noreply.github.com>
@BrendonFaleiro BrendonFaleiro mentioned this pull request Jun 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants