Skip to content

Add log4j.core to requires for the security module#87073

Merged
elasticsearchmachine merged 1 commit intoelastic:masterfrom
ywangd:security-requires-log4j-core
May 24, 2022
Merged

Add log4j.core to requires for the security module#87073
elasticsearchmachine merged 1 commit intoelastic:masterfrom
ywangd:security-requires-log4j-core

Conversation

@ywangd
Copy link
Copy Markdown
Member

@ywangd ywangd commented May 24, 2022

org.apache.logging.log4j.core is referenced in the LoggingAuditTrail
class. This PR adds it to the requires list of the security module.

Relates: #81066

org.apache.logging.log4j.core is referenced by the LoggingAuditTrail
class. This PR adds it to the requires list of the security module.

Relates: elastic#81066
@ywangd ywangd added :Core/Infra/Core Core issues without another label modularization Java Modules related v8.3.0 labels May 24, 2022
@ywangd ywangd requested a review from ChrisHegarty May 24, 2022 07:26
@elasticmachine elasticmachine added the Team:Core/Infra Meta label for core/infra team label May 24, 2022
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-core-infra (Team:Core/Infra)

Copy link
Copy Markdown
Contributor

@ChrisHegarty ChrisHegarty left a comment

Choose a reason for hiding this comment

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

LGTM.

@ywangd ywangd added >non-issue auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) labels May 24, 2022
@elasticsearchmachine elasticsearchmachine merged commit 9644b9f into elastic:master May 24, 2022
@ywangd ywangd deleted the security-requires-log4j-core branch May 24, 2022 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Core/Infra/Core Core issues without another label modularization Java Modules related >non-issue Team:Core/Infra Meta label for core/infra team v8.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants