Skip to content

Assume no operator if security not enabled in serverless#1851

Merged
gbanasiak merged 3 commits intoelastic:masterfrom
gbanasiak:serverless-security-fallback
May 29, 2024
Merged

Assume no operator if security not enabled in serverless#1851
gbanasiak merged 3 commits intoelastic:masterfrom
gbanasiak:serverless-security-fallback

Conversation

@gbanasiak
Copy link
Copy Markdown
Contributor

When testing locally serverless might be running without security enabled. In this case, GET _security/_authenticate fails. This PR falls back to non-operator privileges in such case.

@gbanasiak gbanasiak added this to the 2.11.0 milestone May 29, 2024
@gbanasiak gbanasiak requested a review from gareth-ellis May 29, 2024 12:49
Copy link
Copy Markdown
Member

@gareth-ellis gareth-ellis left a comment

Choose a reason for hiding this comment

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

LGTM

gbanasiak and others added 2 commits May 29, 2024 15:00
Co-authored-by: Quentin Pradet <quentin.pradet@gmail.com>
Copy link
Copy Markdown
Member

@pquentin pquentin left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM.

@gbanasiak gbanasiak merged commit 6706bec into elastic:master May 29, 2024
@gbanasiak gbanasiak deleted the serverless-security-fallback branch May 29, 2024 14:52
@gareth-ellis gareth-ellis added the enhancement Improves the status quo label May 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement Improves the status quo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants