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

Do not overwrite system property net.spy.log.LoggerImpl -> 1.7.1 #1411

Merged
merged 2 commits into from
Jun 11, 2022
Merged
Changes from 1 commit
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Next Next commit
Do not overwrite system property net.spy.log.LoggerImpl
Christophe Maillard authored May 3, 2022

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit 672e3c5e38a81db9ba7588013a4b8625729a73b4
5 changes: 4 additions & 1 deletion framework/src/play/cache/MemcachedImpl.java
Original file line number Diff line number Diff line change
@@ -90,7 +90,10 @@ protected byte[] serialize(Object object) {
}

public void initClient() throws IOException {
System.setProperty("net.spy.log.LoggerImpl", "net.spy.memcached.compat.log.Log4JLogger");
String loggerImpl = System.getProperty("net.spy.log.LoggerImpl");
if (loggerImpl == null) {
System.setProperty("net.spy.log.LoggerImpl", "net.spy.memcached.compat.log.Log4JLogger");
xael-fry marked this conversation as resolved.
Show resolved Hide resolved
}

List<InetSocketAddress> addrs;
if (Play.configuration.containsKey("memcached.host")) {