Skip to content

Implicit jackson config #1

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

Open
iluxa-test wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Implicit jackson config #1

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
bb98ce2
Replace static ObjectMappers with DockerClientConfig#getObjectMapper
bsideup Nov 19, 2019
2c6002e
remove unused `ObjectMapper` field
bsideup Nov 19, 2019
79c3608
revert change
bsideup Nov 19, 2019
cd11cf9
simplify AbstrDockerCmdExec
bsideup Nov 19, 2019
05b4f29
Merge branch 'master' into implicit_jackson_config
bsideup Nov 20, 2019
e9ccbed
Remove unnecessary Jackson annotations
bsideup Nov 20, 2019
1171555
Remove `anUnconfiguredCommandSerializesToEmptyJson` test case
bsideup Nov 20, 2019
72990a2
Try to disable the parallel test execution
bsideup Nov 20, 2019
eebb9ff
Add more logging, try just "hello"
bsideup Nov 20, 2019
1aa68fa
Revert the parallel change
bsideup Nov 20, 2019
0522a79
Back to sequential tests :(
bsideup Nov 20, 2019
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
Prev Previous commit
Next Next commit
revert change
  • Loading branch information
@bsideup
bsideup committed Nov 19, 2019
commit 79c36083b5de11b61174a18e31789b2d85dfbfdd
5 changes: 1 addition & 4 deletions docker-java-core/src/main/java/com/github/dockerjava/core/command/AbstrDockerCmd.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,6 @@
import java.io.IOException;
import java.util.Base64;

import com.fasterxml.jackson.databind.DeserializationFeature;
import org.apache.commons.lang.builder.ReflectionToStringBuilder;
import org.apache.commons.lang.builder.ToStringStyle;
import org.slf4j.Logger;
Expand Down Expand Up @@ -48,9 +47,7 @@ public String toString() {
@Deprecated
protected String registryAuth(AuthConfig authConfig) {
try {
ObjectMapper objectMapper = new ObjectMapper()
.disable(DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES);
return Base64.getEncoder().encodeToString(objectMapper.writeValueAsBytes(authConfig));
return Base64.getEncoder().encodeToString(new ObjectMapper().writeValueAsBytes(authConfig));
} catch (IOException e) {
throw new RuntimeException(e);
}
Expand Down