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

fix(security): explicitly add keys required in security config #6805

Merged

Conversation

shotor
Copy link
Contributor

@shotor shotor commented Mar 28, 2024

Description

The config.dynamic and config.dynamic.authc parts were left out in these files. And it took me hours to realize I was missing those and my auth config wasn't being picked up because of it.

Also if you don't have the _meta key then securityadmin.sh will fail.

I think it's better to add them explicitly, to potentially save someone else some time.

Issues Resolved

Solved 1 of the many issues I'm having in this community post, namely that my authentication config wasn't being picked up at all.

Checklist

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

@shotor shotor force-pushed the fix/explicit-security-config branch from bb85192 to 5ad7dcf Compare March 28, 2024 17:00
@hdhalter
Copy link
Contributor

@scrawfor99 - Can you please check for technical accuracy? Thanks!

@hdhalter hdhalter added 3 - Tech review PR: Tech review in progress backport 2.12 PR: Backport label for 2.12 labels Mar 29, 2024
Copy link
Contributor

@stephen-crawford stephen-crawford left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is just fixing the indentation and layering of the demo docs so looks good.

@shotor shotor requested a review from epugh as a code owner April 1, 2024 19:18
@Naarcha-AWS Naarcha-AWS merged commit 301ca68 into opensearch-project:main Apr 1, 2024
3 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 1, 2024
Signed-off-by: shotor <[email protected]>
Co-authored-by: Naarcha-AWS <[email protected]>
(cherry picked from commit 301ca68)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Naarcha-AWS added a commit that referenced this pull request Apr 2, 2024
#6820)

(cherry picked from commit 301ca68)

Signed-off-by: shotor <[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: Naarcha-AWS <[email protected]>
@shotor shotor deleted the fix/explicit-security-config branch April 2, 2024 19:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Tech review PR: Tech review in progress backport 2.12 PR: Backport label for 2.12
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants