Skip to content

[Backport 2.19] Escape/Unescape pipe UserInfo in ThreadContext#803

Closed
opensearch-trigger-bot[bot] wants to merge 1 commit into2.19from
backport-801-to-2.19
Closed

[Backport 2.19] Escape/Unescape pipe UserInfo in ThreadContext#803
opensearch-trigger-bot[bot] wants to merge 1 commit into2.19from
backport-801-to-2.19

Conversation

@opensearch-trigger-bot
Copy link
Contributor

Backport ef830db from #801.

* Escape pipe character for injected users

Signed-off-by: Shikhar Jain <shikharj@amazon.com>

* test updates + Spotless apply

Signed-off-by: Shikhar Jain <shikharj@amazon.com>

* adding a test case with multiple pipes in username

Signed-off-by: Shikhar Jain <shikharj@amazon.com>

---------

Signed-off-by: Shikhar Jain <shikharj@amazon.com>
Co-authored-by: Shikhar Jain <shikharj@amazon.com>
(cherry picked from commit ef830db)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@shikharj05
Copy link
Contributor

manual backport- #804

@lezzago
Copy link
Member

lezzago commented Mar 20, 2025

closing in favor of #805

@lezzago lezzago closed this Mar 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants