LogRequestor : Replace com.google.common.base.Charsets.UTF_8 with StandardCharsets.UTF_8

[rohanKanojia]

Description

LogRequestor is using Guava's Charset class which has been redundant since a similar class was added in JDK7 : StandardCharsets

docker-maven-plugin/src/main/java/io/fabric8/maven/docker/access/log/LogRequestor.java

Line 173 in 68e5fce

String message = Charsets.UTF_8.newDecoder().decode(payload).toString();

Sonar reports this as a code smell. We should replace usage of this class with JDK provided equivalent.

Acceptance Criteria

• Guava's Charset class is replaced with JDK's StandardCharsets
• LogRequestorTest is also updated with similar change

[rohanKanojia]

@mdxabu : Could you please check if this is clear enough for you? If yes, shall I assign it to you?

[Morbo-ui]

Can I work on this one?

[mdxabu]

@mdxabu : Could you please check if this is clear enough for you? If yes, shall I assign it to you?

Ok, I'll work on that

[rohanKanojia]

@Morbo-ui : Hello, Is it okay if I create some other issue for you? It seems like a PR already got created for this issue.

[Morbo-ui]

@Morbo-ui : Hello, Is it okay if I create some other issue for you? It seems like a PR already got created for this issue.

Yeah no problem, just let me know what can I help with