comment updates

Core/src/com/basistech/df/cybertriage/autopsy/ctapi/CTCloudException.java

@@ -24,11 +24,9 @@ import org.apache.commons.lang3.StringUtils;
 import org.apache.commons.lang3.exception.ExceptionUtils;
 
 /**
- *
+ * An exception thrown due to an error that occurs while making a CT Cloud REST
- * @author rishwanth
+ * API request.
  */
-
-
 public class CTCloudException extends Exception{
     private final ErrorCode errorCode;
     

Core/src/com/basistech/df/cybertriage/autopsy/ctapi/CTCloudHttpClient.java

@@ -73,7 +73,7 @@ import org.apache.http.impl.client.WinHttpClients;
 import org.sleuthkit.autopsy.coreutils.Version;
 
 /**
- * Actually makes the http requests to CT cloud.
+ * Makes the http requests to CT cloud.
  */
 public class CTCloudHttpClient {
 
@@ -263,12 +263,10 @@ public class CTCloudHttpClient {
     }
 
     /**
-     * Creates and returns a CloseableHttpClient SYSTEM and MANUAL looks up from
+     * Creates a connection to CT Cloud with the given arguments.
-     * runtime proxy config settings. These are updated accordingly from the
+     * @param proxySettings The network proxy settings.
-     * Proxy Config UI. This allows us to keep the CreateConnection call fairly
+     * @param sslContext The ssl context or null.
-     * simple and not have to deal with the System Proxy settings and such.
+     * @return The connection to CT Cloud.
-     *
-     * @return
      */
     private static CloseableHttpClient createConnection(ProxySettingArgs proxySettings, SSLContext sslContext) {
         HttpClientBuilder builder = getHttpClientBuilder(proxySettings);

Core/src/com/basistech/df/cybertriage/autopsy/ctapi/Constants.java

@@ -20,6 +20,9 @@ package com.basistech.df.cybertriage.autopsy.ctapi;
 
 import java.net.URI;
 
+/**
+ * Constants regarding connections to cyber triage cloud.
+ */
 final public class Constants {
 
     public static final String CYBER_TRIAGE = "CyberTriage";

Core/src/com/basistech/df/cybertriage/autopsy/ctapi/ProxySettings.java

@@ -18,8 +18,6 @@
  */
 package com.basistech.df.cybertriage.autopsy.ctapi;
 
-
-
 import java.net.*;
 import java.util.*;
 import java.util.logging.Level;

Core/src/com/basistech/df/cybertriage/autopsy/ctapi/util/ObjectMapperUtil.java

@@ -26,8 +26,6 @@ import com.fasterxml.jackson.databind.JsonNode;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.datatype.jsr310.JavaTimeModule;
 import java.io.IOException;
-import java.text.ParseException;
-import java.text.SimpleDateFormat;
 import java.time.DateTimeException;
 import java.time.Instant;
 import java.time.LocalDate;
@@ -37,7 +35,6 @@ import java.time.ZonedDateTime;
 import java.time.format.DateTimeFormatter;
 import java.time.format.DateTimeFormatterBuilder;
 import java.time.format.DateTimeParseException;
-import java.util.Date;
 import java.util.Locale;
 import java.util.function.Function;
 
@@ -58,7 +55,6 @@ public class ObjectMapperUtil {
 
     public ObjectMapper getDefaultObjectMapper() {
         ObjectMapper defaultMapper = new ObjectMapper();
-        // defaultMapper.configure(com.fasterxml.jackson.databind.DeserializationFeature.FAIL_ON_UNKNOWN_PROPERTIES, false);
         defaultMapper.registerModule(new JavaTimeModule());
         return defaultMapper;
     }

Core/src/com/basistech/df/cybertriage/autopsy/ctoptions/CTOptionsPanel.java

@@ -35,7 +35,7 @@ import org.openide.util.Lookup;
 import org.sleuthkit.autopsy.ingest.IngestModuleGlobalSettingsPanel;
 
 /**
- * Options panel for CyberTriage.
+ * Options panel for Cyber Triage.
  */
 public class CTOptionsPanel extends IngestModuleGlobalSettingsPanel {
 
@@ -46,7 +46,8 @@ public class CTOptionsPanel extends IngestModuleGlobalSettingsPanel {
     private final List<CTOptionsSubPanel> subPanels;
 
     /**
-     * Creates new form CTOptions
+     * Creates new form CTOptions loading any CTOptionsSubPanel instances to be
+     * displayed.
      */
     public CTOptionsPanel() {
         initComponents();

Core/src/com/basistech/df/cybertriage/autopsy/ctoptions/ctcloud/CTMalwareScannerOptionsPanel.java

@@ -48,7 +48,8 @@ import org.openide.windows.WindowManager;
 import org.sleuthkit.autopsy.core.UserPreferences;
 
 /**
- * Options panel for CyberTriage options for importing a CyberTriage incident
+ * Options panel to be displayed in the CTOptionsPanel for settings regarding
+ * Cyber Triage Malware Scanner settings and license setup.
  */
 @ServiceProvider(service = CTOptionsSubPanel.class)
 public class CTMalwareScannerOptionsPanel extends CTOptionsSubPanel {
@@ -75,7 +76,7 @@ public class CTMalwareScannerOptionsPanel extends CTOptionsSubPanel {
     private volatile AuthTokenFetcher authTokenFetcher = null;
 
     /**
-     * Creates new form CTIncidentImportOptionsPanel
+     * Main constructor.
      */
     public CTMalwareScannerOptionsPanel() {
         initComponents();

Core/src/com/basistech/df/cybertriage/autopsy/ctoptions/ctcloud/EULADialog.java

@@ -34,8 +34,7 @@ import org.apache.commons.io.IOUtils;
 import org.sleuthkit.autopsy.coreutils.Logger;
 
 /**
- *
+ * Dialog for displaying the Cyber Triage EULA before the license is saved.
- * @author gregd
  */
 public class EULADialog extends javax.swing.JDialog {