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.

Sign up for GitHub

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

Jump to bottom

fix: [CO-863] Missing log4j class using zmproxypurge #423

Merged
merged 4 commits into from
Jan 10, 2024
Merged

fix: [CO-863] Missing log4j class using zmproxypurge #423

merged 4 commits into from
Jan 10, 2024

Conversation

keshavbhatt
Copy link
Member

What has changed:

  • use SLF4JLogger logger Implementation for logging spymemcached
  • allow overriding LoggerImpl for spymemcached using system property(fallsback to SLF4JLogger if not defined)
  • bump spymemcached from version 2.12.1 to 2.12.3

@keshavbhatt keshavbhatt marked this pull request as ready for review January 9, 2024 08:30
@keshavbhatt keshavbhatt requested a review from a team as a code owner January 9, 2024 08:30
@keshavbhatt keshavbhatt requested review from mbarto, danielemegna and mornikco and removed request for a team January 9, 2024 08:30
@keshavbhatt keshavbhatt merged commit ed44ba0 into devel Jan 10, 2024
2 checks passed
@keshavbhatt keshavbhatt deleted the CO-863 branch January 10, 2024 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matteobaglini matteobaglini matteobaglini approved these changes

@zextrasGiovanniDefacci zextrasGiovanniDefacci zextrasGiovanniDefacci approved these changes

@mbarto mbarto Awaiting requested review from mbarto mbarto is a code owner automatically assigned from zextras/kc

@danielemegna danielemegna Awaiting requested review from danielemegna danielemegna is a code owner automatically assigned from zextras/kc

@sonersivri sonersivri Awaiting requested review from sonersivri

@frisonisland frisonisland Awaiting requested review from frisonisland

@aheeva-yuliya aheeva-yuliya Awaiting requested review from aheeva-yuliya

@AlessioCoser AlessioCoser Awaiting requested review from AlessioCoser

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@keshavbhatt @matteobaglini @zextrasGiovanniDefacci