Merge pull request kubernetes-client#240 from archcosmo/master

Fix issue kubernetes-client#233

Author: brendandburns

kubernetes/src/main/java/io/kubernetes/client/JSON.java

@@ -126,11 +126,14 @@ public class ByteArrayAdapter extends TypeAdapter<byte[]> {
 
         @Override
         public void write(JsonWriter out, byte[] value) throws IOException {
+            boolean oldHtmlSafe = out.isHtmlSafe();
+            out.setHtmlSafe(false);
             if (value == null) {
                 out.nullValue();
             } else {
                 out.value(ByteString.of(value).base64());
             }
+            out.setHtmlSafe(oldHtmlSafe);
         }
 
         @Override

kubernetes/src/test/java/io/kubernetes/client/JSONTest.java

@@ -0,0 +1,27 @@
+package io.kubernetes.client;
+
+import okio.ByteString;
+import org.junit.Test;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.junit.Assert.*;
+
+public class JSONTest {
+
+    @Test
+    public void testSerializeByteArray() {
+        final JSON json = new JSON();
+        final String plainText = "string that contains '=' when encoded";
+        final String base64String = json.serialize(plainText.getBytes());
+        //serialize returns string surrounded by quotes: "\"[base64]\""
+        final String pureString = base64String.replaceAll("^\"|\"$", "");
+        final ByteString byteStr = ByteString.decodeBase64(pureString);
+
+        //Check encoded to valid base64
+        assertNotNull(byteStr);
+
+        //Check encoded string correctly
+        final String decodedText =new String(byteStr.toByteArray());
+        assertThat(decodedText, is(plainText));
+    }
+}