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.

Sign up for GitHub

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

Jump to bottom

Registering disable user-agent setting with OpenSearch #16052

Merged
merged 2 commits into from
Sep 27, 2024
Merged

Registering disable user-agent setting with OpenSearch #16052

merged 2 commits into from
Sep 27, 2024

Conversation

saratvemulapalli
Copy link
Member

Description

Follow up of PR: #15325
I missed registering the setting with OpenSearch.

Check List

  • Functionality includes testing.
  • [] API changes companion pull request created, if applicable.
  • Public documentation issue/PR created, if applicable.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@github-actions GitHub Actions
Copy link
Contributor

✅ Gradle check result for 7d18f48: SUCCESS

@saratvemulapalli
Copy link
Member Author

@andrross @owaiskazi19 the original reviewers of #15325 . Could you folks take a look

@saratvemulapalli saratvemulapalli added the backport 2.x Backport to 2.x branch label Sep 27, 2024
@saratvemulapalli saratvemulapalli merged commit 78860ac into opensearch-project:main Sep 27, 2024
56 of 57 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Sep 27, 2024
@github-actions
Registering disable user-agent setting with OpenSearch (#16052)
3feb628 
* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
(cherry picked from commit 78860ac)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@opensearch-trigger-bot opensearch-trigger-bot bot mentioned this pull request Sep 27, 2024
Merged
hainenber pushed a commit to hainenber/OpenSearch that referenced this pull request Oct 1, 2024
@saratvemulapalli @hainenber
Registering disable user-agent setting with OpenSearch (opensearch-pr…
8594e1e 
…oject#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
ruai0511 pushed a commit to ruai0511/OpenSearch that referenced this pull request Oct 4, 2024
@saratvemulapalli @ruai0511
Registering disable user-agent setting with OpenSearch (opensearch-pr…
b806c64 
…oject#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
saratvemulapalli added a commit that referenced this pull request Oct 10, 2024
@opensearch-trigger-bot @github-actions @saratvemulapalli
[Backport 2.x] Registering disable user-agent setting with OpenSearch (…
f795fdb 
…#16113)

* Registering disable user-agent setting with OpenSearch (#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
(cherry picked from commit 78860ac)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>

* Fixing backport 2.x

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Sarat Vemulapalli <[email protected]>
@saratvemulapalli saratvemulapalli deleted the useragent-setting branch October 15, 2024 22:48
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 16, 2024
@saratvemulapalli @dk2k
Registering disable user-agent setting with OpenSearch (opensearch-pr…
e2088a0 
…oject#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 17, 2024
@saratvemulapalli @dk2k
Registering disable user-agent setting with OpenSearch (opensearch-pr…
0286102 
…oject#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
dk2k pushed a commit to dk2k/OpenSearch that referenced this pull request Oct 21, 2024
@saratvemulapalli @dk2k
Registering disable user-agent setting with OpenSearch (opensearch-pr…
437b942 
…oject#16052)

* Registering allowlist setting in user-agent

Signed-off-by: Sarat Vemulapalli <[email protected]>

* Adding unit tests

Signed-off-by: Sarat Vemulapalli <[email protected]>

---------

Signed-off-by: Sarat Vemulapalli <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@owaiskazi19 owaiskazi19 owaiskazi19 approved these changes

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz is a code owner

@andrross andrross Awaiting requested review from andrross andrross is a code owner

@ashking94 ashking94 Awaiting requested review from ashking94 ashking94 is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@dblock dblock Awaiting requested review from dblock dblock is a code owner

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@jainankitk jainankitk Awaiting requested review from jainankitk jainankitk is a code owner

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal is a code owner

@linuxpi linuxpi Awaiting requested review from linuxpi linuxpi is a code owner

@mch2 mch2 Awaiting requested review from mch2 mch2 is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh is a code owner

@nknize nknize Awaiting requested review from nknize nknize is a code owner

@reta reta Awaiting requested review from reta reta is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 is a code owner

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami is a code owner

@VachaShah VachaShah Awaiting requested review from VachaShah VachaShah is a code owner

Assignees
No one assigned
Labels
backport 2.x Backport to 2.x branch skip-changelog
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@saratvemulapalli @owaiskazi19