finish 0.31

fix bug where null har content text would cause HarCleaner exception

Author: mike10004

src/main/java/com/github/mike10004/seleniumhelp/HarCleaner.java

@@ -48,7 +48,7 @@ public List<HarEntry> clean(Har inputHar) {
             String contentEncoding = getHeaderValue(rsp.getHeaders(), HttpHeaders.CONTENT_ENCODING);
             if (HEADER_VALUE_BROTLI_ENCODED.equalsIgnoreCase(contentEncoding) && content != null) {
                 String text = content.getText();
-                if (base64Alphabet().matchesAllOf(text)) {
+                if (text != null && base64Alphabet().matchesAllOf(text)) {
                     return true;
                 }
             }

src/test/java/com/github/mike10004/seleniumhelp/HarCleanerTest.java

@@ -4,6 +4,7 @@
 import com.google.common.io.CharSource;
 import com.google.common.io.Resources;
 import com.google.common.net.HttpHeaders;
+import com.google.common.net.MediaType;
 import net.lightbody.bmp.core.har.*;
 import org.junit.Test;
 
@@ -95,4 +96,17 @@ private HarResponse buildResponse(String contentTypeHeader, String contentEncodi
         response.getHeaders().addAll(headers);
         return response;
     }
+
+    @Test
+    public void clean_entryResponseContentTextNull() throws Exception {
+        Har har = new Har();
+        HarResponse response = buildResponse(MediaType.JAVASCRIPT_UTF_8.toString(), "br", (String)null, "base64");
+        HarEntry entry = new HarEntry();
+        entry.setResponse(response);
+        HarLog log = new HarLog();
+        log.addEntry(entry);
+        har.setLog(log);
+        new HarCleaner().clean(har);
+        assertEquals("num entries after clean", 1, har.getLog().getEntries().size());
+    }
 }