Skip to content

Add condition field to Keycloak log datastream #11213

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

Merged
merged 4 commits into from
Sep 26, 2024
Merged

Add condition field to Keycloak log datastream #11213

merged 4 commits into from
Sep 26, 2024

Conversation

Alphayeeeet
Copy link
Contributor

Please label as Enhancement

This PR adds the concition field as configurable option to the Keycloak log datastream

@Alphayeeeet Alphayeeeet requested a review from a team as a code owner September 23, 2024 15:46
@andrewkroh andrewkroh added bugfix Pull request that fixes a bug issue Integration:keycloak Keycloak (Community supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] labels Sep 23, 2024
@elasticmachine
Copy link

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@Alphayeeeet
Copy link
Contributor Author

@andrewkroh FYI this PR is not a bugfix, but rather an enhancement.

@andrewkroh andrewkroh added enhancement New feature or request and removed bugfix Pull request that fixes a bug issue labels Sep 23, 2024
@Alphayeeeet
Copy link
Contributor Author

Alphayeeeet commented Sep 24, 2024

@andrewkroh Could you please run CI again?

@andrewkroh
Copy link
Member

/test

@Alphayeeeet
Copy link
Contributor Author

@andrewkroh Package tests look alright, but CI has a problem when stopping test environment. Is this an issue of the integration or of the CI environment?

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
@andrewkroh
Copy link
Member

/test

@andrewkroh
Copy link
Member

It's a CI issue. I am told that #11227 is testing a fix.

@andrewkroh
Copy link
Member

/test

@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

History

Copy link

@andrewkroh andrewkroh merged commit 5205b4a into elastic:main Sep 26, 2024
5 checks passed
@elastic-vault-github-plugin-prod

Package keycloak - 1.23.0 containing this change is available at https://epr.elastic.co/search?package=keycloak

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
Make condition configurable to control when to apply the log data stream.
The elastic agent evaluates the condition to determine whether or not to
execute the input.
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
Make condition configurable to control when to apply the log data stream.
The elastic agent evaluates the condition to determine whether or not to
execute the input.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request Integration:keycloak Keycloak (Community supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants