Refactor Prov DB handling
[dmaap/datarouter.git] / datarouter-prov / src / main / java / org / onap / dmaap / datarouter / provisioning / beans / Subscription.java
index 230df75..5741881 100644 (file)
 \r
 package org.onap.dmaap.datarouter.provisioning.beans;\r
 \r
+import com.att.eelf.configuration.EELFLogger;\r
+import com.att.eelf.configuration.EELFManager;\r
 import java.io.InvalidObjectException;\r
 import java.sql.Connection;\r
 import java.sql.PreparedStatement;\r
 import java.sql.ResultSet;\r
 import java.sql.SQLException;\r
-import java.sql.Statement;\r
 import java.util.ArrayList;\r
 import java.util.Collection;\r
 import java.util.Date;\r
 import java.util.List;\r
-import java.util.Objects;\r
 import java.util.Properties;\r
-import org.apache.log4j.Logger;\r
 import org.json.JSONObject;\r
-import org.onap.dmaap.datarouter.provisioning.utils.DB;\r
+import org.onap.dmaap.datarouter.provisioning.ProvRunner;\r
+import org.onap.dmaap.datarouter.provisioning.utils.ProvDbUtils;\r
 import org.onap.dmaap.datarouter.provisioning.utils.URLUtilities;\r
 \r
+\r
 /**\r
  * The representation of a Subscription.  Subscriptions can be retrieved from the DB, or stored/updated in the DB.\r
  *\r
  * @author Robert Eby\r
  * @version $Id: Subscription.java,v 1.9 2013/10/28 18:06:53 eby Exp $\r
  */\r
+\r
 public class Subscription extends Syncable {\r
 \r
     private static final String SQLEXCEPTION = "SQLException: ";\r
@@ -55,13 +57,14 @@ public class Subscription extends Syncable {
     private static final String GROUPID_KEY = "groupid";\r
     private static final String LAST_MOD_KEY = "last_mod";\r
     private static final String CREATED_DATE = "created_date";\r
-    private static Logger intlogger = Logger.getLogger("org.onap.dmaap.datarouter.provisioning.internal");\r
+    private static EELFLogger intlogger = EELFManager.getInstance().getLogger("InternalLog");\r
     private static int nextSubid = getMaxSubID() + 1;\r
 \r
     private int subid;\r
     private int feedid;\r
     private int groupid; //New field is added - Groups feature Rally:US708115 - 1610\r
     private SubDelivery delivery;\r
+    private boolean followRedirect;\r
     private boolean metadataOnly;\r
     private String subscriber;\r
     private SubLinks links;\r
@@ -69,156 +72,64 @@ public class Subscription extends Syncable {
     private Date lastMod;\r
     private Date createdDate;\r
     private boolean privilegedSubscriber;\r
-\r
-    public static Subscription getSubscriptionMatching(Subscription sub) {\r
-        SubDelivery deli = sub.getDelivery();\r
-        String sql = String.format(\r
-                "select * from SUBSCRIPTIONS where FEEDID = %d and DELIVERY_URL = \"%s\" and DELIVERY_USER = \"%s\" and DELIVERY_PASSWORD = \"%s\" and DELIVERY_USE100 = %d and METADATA_ONLY = %d",\r
-                sub.getFeedid(),\r
-                deli.getUrl(),\r
-                deli.getUser(),\r
-                deli.getPassword(),\r
-                deli.isUse100() ? 1 : 0,\r
-                sub.isMetadataOnly() ? 1 : 0\r
-        );\r
-        List<Subscription> list = getSubscriptionsForSQL(sql);\r
-        return !list.isEmpty() ? list.get(0) : null;\r
-    }\r
-\r
-    public static Subscription getSubscriptionById(int id) {\r
-        String sql = "select * from SUBSCRIPTIONS where SUBID = " + id;\r
-        List<Subscription> list = getSubscriptionsForSQL(sql);\r
-        return !list.isEmpty() ? list.get(0) : null;\r
-    }\r
-\r
-    public static Collection<Subscription> getAllSubscriptions() {\r
-        return getSubscriptionsForSQL("select * from SUBSCRIPTIONS");\r
-    }\r
-\r
-    private static List<Subscription> getSubscriptionsForSQL(String sql) {\r
-        List<Subscription> list = new ArrayList<>();\r
-        try {\r
-            DB db = new DB();\r
-            @SuppressWarnings("resource")\r
-            Connection conn = db.getConnection();\r
-            try (Statement stmt = conn.createStatement()) {\r
-                try (ResultSet rs = stmt.executeQuery(sql)) {\r
-                    while (rs.next()) {\r
-                        Subscription sub = new Subscription(rs);\r
-                        list.add(sub);\r
-                    }\r
-                }\r
-            }\r
-            db.release(conn);\r
-        } catch (SQLException e) {\r
-            intlogger.error(e);\r
-        }\r
-        return list;\r
-    }\r
-\r
-    public static int getMaxSubID() {\r
-        int max = 0;\r
-        try {\r
-            DB db = new DB();\r
-            @SuppressWarnings("resource")\r
-            Connection conn = db.getConnection();\r
-            try (Statement stmt = conn.createStatement()) {\r
-                try (ResultSet rs = stmt.executeQuery("select MAX(subid) from SUBSCRIPTIONS")) {\r
-                    if (rs.next()) {\r
-                        max = rs.getInt(1);\r
-                    }\r
-                }\r
-            }\r
-            db.release(conn);\r
-        } catch (SQLException e) {\r
-            intlogger.info("getMaxSubID: " + e.getMessage());\r
-        }\r
-        return max;\r
-    }\r
-\r
-    public static Collection<String> getSubscriptionUrlList(int feedid) {\r
-        List<String> list = new ArrayList<>();\r
-        String sql = "select SUBID from SUBSCRIPTIONS where FEEDID = ?";\r
-\r
-        try {\r
-            DB db = new DB();\r
-            @SuppressWarnings("resource")\r
-            Connection conn = db.getConnection();\r
-\r
-            try (PreparedStatement stmt = conn.prepareStatement(sql)) {\r
-                stmt.setString(1, String.valueOf(feedid));\r
-                try (ResultSet rs = stmt.executeQuery()) {\r
-                    while (rs.next()) {\r
-                        int subid = rs.getInt(SUBID_COL);\r
-                        list.add(URLUtilities.generateSubscriptionURL(subid));\r
-                    }\r
-                }\r
-            }\r
-            db.release(conn);\r
-        } catch (SQLException e) {\r
-            intlogger.error(SQLEXCEPTION + e.getMessage());\r
-        }\r
-        return list;\r
-    }\r
-\r
-    /**\r
-     * Return a count of the number of active subscriptions in the DB.\r
-     *\r
-     * @return the count\r
-     */\r
-    public static int countActiveSubscriptions() {\r
-        int count = 0;\r
-        try {\r
-            DB db = new DB();\r
-            @SuppressWarnings("resource")\r
-            Connection conn = db.getConnection();\r
-            try (Statement stmt = conn.createStatement()) {\r
-                try (ResultSet rs = stmt.executeQuery("select count(*) from SUBSCRIPTIONS")) {\r
-                    if (rs.next()) {\r
-                        count = rs.getInt(1);\r
-                    }\r
-                }\r
-            }\r
-            db.release(conn);\r
-        } catch (SQLException e) {\r
-            intlogger.warn("PROV0008 countActiveSubscriptions: " + e.getMessage());\r
-        }\r
-        return count;\r
-    }\r
+    private String aafInstance;\r
+    private boolean decompress;\r
 \r
     public Subscription() {\r
         this("", "", "");\r
     }\r
 \r
+    /**\r
+     * Subscription constructor.\r
+     * @param url url string\r
+     * @param user user string\r
+     * @param password password string\r
+     */\r
     public Subscription(String url, String user, String password) {\r
         this.subid = -1;\r
         this.feedid = -1;\r
         this.groupid = -1; //New field is added - Groups feature Rally:US708115 - 1610\r
         this.delivery = new SubDelivery(url, user, password, false);\r
         this.metadataOnly = false;\r
+        this.followRedirect = false;\r
         this.subscriber = "";\r
         this.links = new SubLinks();\r
         this.suspended = false;\r
         this.lastMod = new Date();\r
         this.createdDate = new Date();\r
         this.privilegedSubscriber = false;\r
+        this.aafInstance = "";\r
+        this.decompress = false;\r
     }\r
 \r
+    /**\r
+     * Subscription constructor.\r
+     * @param rs resultset from SQL\r
+     * @throws SQLException in case of SQL error\r
+     */\r
     public Subscription(ResultSet rs) throws SQLException {\r
         this.subid = rs.getInt(SUBID_COL);\r
         this.feedid = rs.getInt("FEEDID");\r
         this.groupid = rs.getInt("GROUPID"); //New field is added - Groups feature Rally:US708115 - 1610\r
         this.delivery = new SubDelivery(rs);\r
         this.metadataOnly = rs.getBoolean("METADATA_ONLY");\r
+        this.followRedirect = rs.getBoolean("FOLLOW_REDIRECTS");\r
         this.subscriber = rs.getString("SUBSCRIBER");\r
         this.links = new SubLinks(rs.getString("SELF_LINK"), URLUtilities.generateFeedURL(feedid),\r
-                rs.getString("LOG_LINK"));\r
+            rs.getString("LOG_LINK"));\r
         this.suspended = rs.getBoolean("SUSPENDED");\r
         this.lastMod = rs.getDate("LAST_MOD");\r
         this.createdDate = rs.getDate("CREATED_DATE");\r
         this.privilegedSubscriber = rs.getBoolean("PRIVILEGED_SUBSCRIBER");\r
+        this.aafInstance = rs.getString("AAF_INSTANCE");\r
+        this.decompress  = rs.getBoolean("DECOMPRESS");\r
     }\r
 \r
+    /**\r
+     * Subscription constructor.\r
+     * @param jo JSONObject\r
+     * @throws InvalidObjectException in case of object error\r
+     */\r
     public Subscription(JSONObject jo) throws InvalidObjectException {\r
         this("", "", "");\r
         try {\r
@@ -226,52 +137,178 @@ public class Subscription extends Syncable {
             this.subid = jo.optInt(SUBID_KEY, -1);\r
             this.feedid = jo.optInt(FEEDID_KEY, -1);\r
             this.groupid = jo.optInt(GROUPID_KEY, -1); //New field is added - Groups feature Rally:US708115 - 1610\r
-\r
+            this.aafInstance = jo.optString("aaf_instance", "legacy");\r
+            if (!(aafInstance.equalsIgnoreCase("legacy")) && aafInstance.length() > 255) {\r
+                throw new InvalidObjectException("aaf_instance field is too long");\r
+            }\r
             JSONObject jdeli = jo.getJSONObject("delivery");\r
             String url = jdeli.getString("url");\r
             String user = jdeli.getString("user");\r
-            String password = jdeli.getString("password");\r
-            boolean use100 = jdeli.getBoolean("use100");\r
+            final String password = jdeli.getString("password");\r
+            final boolean use100 = jdeli.getBoolean("use100");\r
 \r
             //Data Router Subscriber HTTPS Relaxation feature USERSTORYID:US674047.\r
-            Properties p = (new DB()).getProperties();\r
-            if (!url.startsWith("https://") && isHttpsRelaxationFalseAndHasSyncKey(jo, p)) {\r
+            Properties prop = ProvRunner.getProvProperties();\r
+            if (!url.startsWith("https://") && isHttpsRelaxationFalseAndHasSyncKey(jo, prop)) {\r
                 throw new InvalidObjectException("delivery URL is not HTTPS");\r
             }\r
 \r
             if (url.length() > 256) {\r
                 throw new InvalidObjectException("delivery url field is too long");\r
             }\r
-            if (user.length() > 20) {\r
+            if (user.length() > 60) {\r
                 throw new InvalidObjectException("delivery user field is too long");\r
             }\r
             if (password.length() > 32) {\r
                 throw new InvalidObjectException("delivery password field is too long");\r
             }\r
             this.delivery = new SubDelivery(url, user, password, use100);\r
-\r
             this.metadataOnly = jo.getBoolean("metadataOnly");\r
+            this.followRedirect = jo.optBoolean("follow_redirect", false);\r
             this.suspended = jo.optBoolean("suspend", false);\r
             this.privilegedSubscriber = jo.optBoolean("privilegedSubscriber", false);\r
+            this.decompress = jo.optBoolean("decompress", false);\r
             this.subscriber = jo.optString("subscriber", "");\r
             JSONObject jol = jo.optJSONObject("links");\r
             this.links = (jol == null) ? (new SubLinks()) : (new SubLinks(jol));\r
         } catch (InvalidObjectException e) {\r
             throw e;\r
         } catch (Exception e) {\r
-            throw new InvalidObjectException("invalid JSON: " + e.getMessage());\r
+            intlogger.warn("Invalid JSON: " + e.getMessage(), e);\r
+            throw new InvalidObjectException("Invalid JSON: " + e.getMessage());\r
         }\r
     }\r
 \r
-    private boolean isHttpsRelaxationFalseAndHasSyncKey(JSONObject jo, Properties p) {\r
-        return p.get("org.onap.dmaap.datarouter.provserver.https.relaxation").toString().equals("false") && !jo\r
-                .has("sync");\r
+    /**\r
+     * Get specific subscription.\r
+     * @param sub subscription object\r
+     * @return subscription\r
+     */\r
+    public static Subscription getSubscriptionMatching(Subscription sub) {\r
+        SubDelivery deli = sub.getDelivery();\r
+        String sql = String.format(\r
+            "select * from SUBSCRIPTIONS where FEEDID = %d and DELIVERY_URL = \"%s\" and DELIVERY_USER = \"%s\" "\r
+                + "and DELIVERY_PASSWORD = \"%s\" and DELIVERY_USE100 = %d and METADATA_ONLY = %d "\r
+                + "and FOLLOW_REDIRECTS = %d",\r
+            sub.getFeedid(),\r
+            deli.getUrl(),\r
+            deli.getUser(),\r
+            deli.getPassword(),\r
+            deli.isUse100() ? 1 : 0,\r
+            sub.isMetadataOnly() ? 1 : 0,\r
+            sub.isFollowRedirect() ? 1 : 0\r
+        );\r
+        List<Subscription> list = getSubscriptionsForSQL(sql);\r
+        return !list.isEmpty() ? list.get(0) : null;\r
+    }\r
+\r
+    /**\r
+     * Get subscription by id.\r
+     * @param id subscription id string\r
+     * @return subscription\r
+     */\r
+    public static Subscription getSubscriptionById(int id) {\r
+        String sql = "select * from SUBSCRIPTIONS where SUBID = " + id;\r
+        List<Subscription> list = getSubscriptionsForSQL(sql);\r
+        return !list.isEmpty() ? list.get(0) : null;\r
+    }\r
+\r
+    public static Collection<Subscription> getAllSubscriptions() {\r
+        return getSubscriptionsForSQL("select * from SUBSCRIPTIONS");\r
+    }\r
+\r
+    /**\r
+     * Get subscriptions from SQL.\r
+     * @param sql SQL statement\r
+     * @return List of subscriptions\r
+     */\r
+    private static List<Subscription> getSubscriptionsForSQL(String sql) {\r
+        List<Subscription> list = new ArrayList<>();\r
+        try (Connection conn = ProvDbUtils.getInstance().getConnection();\r
+            PreparedStatement ps = conn.prepareStatement(sql);\r
+            ResultSet rs = ps.executeQuery()) {\r
+            while (rs.next()) {\r
+                Subscription sub = new Subscription(rs);\r
+                list.add(sub);\r
+            }\r
+        } catch (SQLException e) {\r
+            intlogger.error("PROV0001 getSubscriptionsForSQL: " + e.toString(), e);\r
+        }\r
+        return list;\r
+    }\r
+\r
+    /**\r
+     * Get max subid.\r
+     * @return subid int\r
+     */\r
+    public static int getMaxSubID() {\r
+        int max = 0;\r
+        try (Connection conn = ProvDbUtils.getInstance().getConnection();\r
+            PreparedStatement ps = conn.prepareStatement("select MAX(subid) from SUBSCRIPTIONS");\r
+            ResultSet rs = ps.executeQuery()) {\r
+            if (rs.next()) {\r
+                max = rs.getInt(1);\r
+            }\r
+        } catch (SQLException e) {\r
+            intlogger.info("getMaxSubID: " + e.getMessage(), e);\r
+        }\r
+        return max;\r
+    }\r
+\r
+    /**\r
+     * Get subscription URL list.\r
+     * @param feedid feedid int\r
+     * @return collection of subscription URL\r
+     */\r
+    public static Collection<String> getSubscriptionUrlList(int feedid) {\r
+        List<String> list = new ArrayList<>();\r
+        try (Connection conn = ProvDbUtils.getInstance().getConnection();\r
+            PreparedStatement stmt = conn.prepareStatement("select SUBID from SUBSCRIPTIONS where FEEDID = ?")) {\r
+            stmt.setString(1, String.valueOf(feedid));\r
+            try (ResultSet rs = stmt.executeQuery()) {\r
+                while (rs.next()) {\r
+                    int subid = rs.getInt(SUBID_COL);\r
+                    list.add(URLUtilities.generateSubscriptionURL(subid));\r
+                }\r
+            }\r
+        } catch (SQLException e) {\r
+            intlogger.error(SQLEXCEPTION + e.getMessage(), e);\r
+        }\r
+        return list;\r
+    }\r
+\r
+    /**\r
+     * Return a count of the number of active subscriptions in the DB.\r
+     *\r
+     * @return the count\r
+     */\r
+    public static int countActiveSubscriptions() {\r
+        int count = 0;\r
+        try (Connection conn = ProvDbUtils.getInstance().getConnection();\r
+            PreparedStatement ps = conn.prepareStatement("select count(*) from SUBSCRIPTIONS");\r
+            ResultSet rs = ps.executeQuery()) {\r
+            if (rs.next()) {\r
+                count = rs.getInt(1);\r
+            }\r
+        } catch (SQLException e) {\r
+            intlogger.warn("PROV0008 countActiveSubscriptions: " + e.getMessage(), e);\r
+        }\r
+        return count;\r
+    }\r
+\r
+    private boolean isHttpsRelaxationFalseAndHasSyncKey(JSONObject jo, Properties prop) {\r
+        return prop.get("org.onap.dmaap.datarouter.provserver.https.relaxation").toString().equals("false") && !jo\r
+            .has("sync");\r
     }\r
 \r
     public int getSubid() {\r
         return subid;\r
     }\r
 \r
+    /**\r
+     * Subid setter.\r
+     * @param subid subid string\r
+     */\r
     public void setSubid(int subid) {\r
         this.subid = subid;\r
 \r
@@ -285,6 +322,10 @@ public class Subscription extends Syncable {
         return feedid;\r
     }\r
 \r
+    /**\r
+     * feedid setter.\r
+     * @param feedid feedid string\r
+     */\r
     public void setFeedid(int feedid) {\r
         this.feedid = feedid;\r
 \r
@@ -293,6 +334,14 @@ public class Subscription extends Syncable {
         sl.setFeed(URLUtilities.generateFeedURL(feedid));\r
     }\r
 \r
+    public String getAafInstance() {\r
+        return aafInstance;\r
+    }\r
+\r
+    public void setAafInstance(String aafInstance) {\r
+        this.aafInstance = aafInstance;\r
+    }\r
+\r
     //New getter setters for Groups feature Rally:US708115 - 1610\r
     public int getGroupid() {\r
         return groupid;\r
@@ -318,7 +367,15 @@ public class Subscription extends Syncable {
         this.metadataOnly = metadataOnly;\r
     }\r
 \r
-    public boolean isSuspended() {\r
+    private boolean isFollowRedirect() {\r
+        return followRedirect;\r
+    }\r
+\r
+    public void setFollowRedirect(boolean followRedirect) {\r
+        this.followRedirect = followRedirect;\r
+    }\r
+\r
+    boolean isSuspended() {\r
         return suspended;\r
     }\r
 \r
@@ -338,6 +395,10 @@ public class Subscription extends Syncable {
         return subscriber;\r
     }\r
 \r
+    /**\r
+     * Subscriber setter.\r
+     * @param subscriber subscriber string\r
+     */\r
     public void setSubscriber(String subscriber) {\r
         if (subscriber != null) {\r
             if (subscriber.length() > 8) {\r
@@ -351,10 +412,18 @@ public class Subscription extends Syncable {
         return links;\r
     }\r
 \r
-    public void setLinks(SubLinks links) {\r
+    void setLinks(SubLinks links) {\r
         this.links = links;\r
     }\r
 \r
+    public boolean isDecompress() {\r
+        return decompress;\r
+    }\r
+\r
+    public void setDecompress(boolean decompress) {\r
+        this.decompress = decompress;\r
+    }\r
+\r
     @Override\r
     public JSONObject asJSONObject() {\r
         JSONObject jo = new JSONObject();\r
@@ -363,23 +432,23 @@ public class Subscription extends Syncable {
         jo.put(GROUPID_KEY, groupid); //New field is added - Groups feature Rally:US708115 - 1610\r
         jo.put("delivery", delivery.asJSONObject());\r
         jo.put("metadataOnly", metadataOnly);\r
+        jo.put("follow_redirect", followRedirect);\r
         jo.put("subscriber", subscriber);\r
         jo.put("links", links.asJSONObject());\r
         jo.put("suspend", suspended);\r
         jo.put(LAST_MOD_KEY, lastMod.getTime());\r
         jo.put(CREATED_DATE, createdDate.getTime());\r
         jo.put("privilegedSubscriber", privilegedSubscriber);\r
+        jo.put("aaf_instance", aafInstance);\r
+        jo.put("decompress", decompress);\r
         return jo;\r
     }\r
 \r
-    public JSONObject asLimitedJSONObject() {\r
-        JSONObject jo = asJSONObject();\r
-        jo.remove(SUBID_KEY);\r
-        jo.remove(FEEDID_KEY);\r
-        jo.remove(LAST_MOD_KEY);\r
-        return jo;\r
-    }\r
-\r
+    /**\r
+     * Method to hide attributes.\r
+     * @param hidepasswords true/false\r
+     * @return JSONObject\r
+     */\r
     public JSONObject asJSONObject(boolean hidepasswords) {\r
         JSONObject jo = asJSONObject();\r
         if (hidepasswords) {\r
@@ -391,8 +460,21 @@ public class Subscription extends Syncable {
         return jo;\r
     }\r
 \r
+    /**\r
+     * Method to remove some attributes from JSON.\r
+     * @ JSONObject\r
+     */\r
+    public JSONObject asLimitedJSONObject() {\r
+        JSONObject jo = asJSONObject();\r
+        jo.remove(SUBID_KEY);\r
+        jo.remove(FEEDID_KEY);\r
+        jo.remove(LAST_MOD_KEY);\r
+        return jo;\r
+    }\r
+\r
+\r
     @Override\r
-    public boolean doInsert(Connection c) {\r
+    public boolean doInsert(Connection conn) {\r
         boolean rv = true;\r
         PreparedStatement ps = null;\r
         try {\r
@@ -406,8 +488,11 @@ public class Subscription extends Syncable {
             }\r
 \r
             // Create the SUBSCRIPTIONS row\r
-            String sql = "insert into SUBSCRIPTIONS (SUBID, FEEDID, DELIVERY_URL, DELIVERY_USER, DELIVERY_PASSWORD, DELIVERY_USE100, METADATA_ONLY, SUBSCRIBER, SUSPENDED, GROUPID, PRIVILEGED_SUBSCRIBER) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";\r
-            ps = c.prepareStatement(sql, new String[]{SUBID_COL});\r
+            String sql = "insert into SUBSCRIPTIONS (SUBID, FEEDID, DELIVERY_URL, DELIVERY_USER, DELIVERY_PASSWORD, "\r
+                + "DELIVERY_USE100, METADATA_ONLY, SUBSCRIBER, SUSPENDED, GROUPID, "\r
+                + "PRIVILEGED_SUBSCRIBER, FOLLOW_REDIRECTS, DECOMPRESS, AAF_INSTANCE) "\r
+                + "values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";\r
+            ps = conn.prepareStatement(sql, new String[]{SUBID_COL});\r
             ps.setInt(1, subid);\r
             ps.setInt(2, feedid);\r
             ps.setString(3, getDelivery().getUrl());\r
@@ -419,11 +504,14 @@ public class Subscription extends Syncable {
             ps.setBoolean(9, isSuspended());\r
             ps.setInt(10, groupid); //New field is added - Groups feature Rally:US708115 - 1610\r
             ps.setBoolean(11, isPrivilegedSubscriber());\r
+            ps.setInt(12, isFollowRedirect() ? 1 : 0);\r
+            ps.setBoolean(13, isDecompress());\r
+            ps.setString(14, getAafInstance());\r
             ps.execute();\r
             ps.close();\r
             // Update the row to set the URLs\r
             sql = "update SUBSCRIPTIONS set SELF_LINK = ?, LOG_LINK = ? where SUBID = ?";\r
-            ps = c.prepareStatement(sql);\r
+            ps = conn.prepareStatement(sql);\r
             ps.setString(1, getLinks().getSelf());\r
             ps.setString(2, getLinks().getLog());\r
             ps.setInt(3, subid);\r
@@ -431,26 +519,26 @@ public class Subscription extends Syncable {
             ps.close();\r
         } catch (SQLException e) {\r
             rv = false;\r
-            intlogger.warn("PROV0005 doInsert: " + e.getMessage());\r
+            intlogger.warn("PROV0005 doInsert: " + e.getMessage(), e);\r
         } finally {\r
             try {\r
                 if (ps != null) {\r
                     ps.close();\r
                 }\r
             } catch (SQLException e) {\r
-                intlogger.error(SQLEXCEPTION + e.getMessage());\r
+                intlogger.error(SQLEXCEPTION + e.getMessage(), e);\r
             }\r
         }\r
         return rv;\r
     }\r
 \r
     @Override\r
-    public boolean doUpdate(Connection c) {\r
+    public boolean doUpdate(Connection conn) {\r
         boolean rv = true;\r
-        PreparedStatement ps = null;\r
-        try {\r
-            String sql = "update SUBSCRIPTIONS set DELIVERY_URL = ?, DELIVERY_USER = ?, DELIVERY_PASSWORD = ?, DELIVERY_USE100 = ?, METADATA_ONLY = ?, SUSPENDED = ?, GROUPID = ?, PRIVILEGED_SUBSCRIBER = ? where SUBID = ?";\r
-            ps = c.prepareStatement(sql);\r
+        try (PreparedStatement ps = conn.prepareStatement(\r
+            "update SUBSCRIPTIONS set DELIVERY_URL = ?, DELIVERY_USER = ?, DELIVERY_PASSWORD = ?, "\r
+            + "DELIVERY_USE100 = ?, METADATA_ONLY = ?, SUSPENDED = ?, GROUPID = ?, PRIVILEGED_SUBSCRIBER = ?, "\r
+            + "FOLLOW_REDIRECTS = ?, DECOMPRESS = ? where SUBID = ?")) {\r
             ps.setString(1, delivery.getUrl());\r
             ps.setString(2, delivery.getUser());\r
             ps.setString(3, delivery.getPassword());\r
@@ -459,77 +547,46 @@ public class Subscription extends Syncable {
             ps.setInt(6, suspended ? 1 : 0);\r
             ps.setInt(7, groupid); //New field is added - Groups feature Rally:US708115 - 1610\r
             ps.setInt(8, privilegedSubscriber ? 1 : 0);\r
-            ps.setInt(9, subid);\r
+            ps.setInt(9, isFollowRedirect() ? 1 : 0);\r
+            ps.setInt(10, isDecompress() ? 1 : 0);\r
+            ps.setInt(11, subid);\r
             ps.executeUpdate();\r
         } catch (SQLException e) {\r
             rv = false;\r
-            intlogger.warn("PROV0006 doUpdate: " + e.getMessage());\r
-        } finally {\r
-            try {\r
-                if (ps != null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                intlogger.error(SQLEXCEPTION + e.getMessage());\r
-            }\r
+            intlogger.warn("PROV0006 doUpdate: " + e.getMessage(), e);\r
         }\r
         return rv;\r
     }\r
 \r
 \r
     /**\r
-     * Rally US708115 Change Ownership of Subscription - 1610\r
+     * Rally US708115 Change Ownership of Subscription - 1610.\r
      */\r
     public boolean changeOwnerShip() {\r
         boolean rv = true;\r
-        PreparedStatement ps = null;\r
-        try {\r
-\r
-            DB db = new DB();\r
-            @SuppressWarnings("resource")\r
-            Connection c = db.getConnection();\r
-            String sql = "update SUBSCRIPTIONS set SUBSCRIBER = ? where SUBID = ?";\r
-            ps = c.prepareStatement(sql);\r
+        try (Connection conn = ProvDbUtils.getInstance().getConnection();\r
+            PreparedStatement ps = conn.prepareStatement(\r
+                "update SUBSCRIPTIONS set SUBSCRIBER = ? where SUBID = ?")) {\r
             ps.setString(1, this.subscriber);\r
             ps.setInt(2, subid);\r
             ps.execute();\r
-            ps.close();\r
         } catch (SQLException e) {\r
             rv = false;\r
-            intlogger.warn("PROV0006 doUpdate: " + e.getMessage());\r
-        } finally {\r
-            try {\r
-                if (ps != null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                intlogger.error(SQLEXCEPTION + e.getMessage());\r
-            }\r
+            intlogger.warn("PROV0006 doUpdate: " + e.getMessage(), e);\r
         }\r
         return rv;\r
     }\r
 \r
 \r
     @Override\r
-    public boolean doDelete(Connection c) {\r
+    public boolean doDelete(Connection conn) {\r
         boolean rv = true;\r
-        PreparedStatement ps = null;\r
-        try {\r
-            String sql = "delete from SUBSCRIPTIONS where SUBID = ?";\r
-            ps = c.prepareStatement(sql);\r
+        try (PreparedStatement ps = conn.prepareStatement("delete from SUBSCRIPTIONS where SUBID = ?")) {\r
             ps.setInt(1, subid);\r
             ps.execute();\r
         } catch (SQLException e) {\r
             rv = false;\r
-            intlogger.warn("PROV0007 doDelete: " + e.getMessage());\r
-        } finally {\r
-            try {\r
-                if (ps != null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                intlogger.error(SQLEXCEPTION + e.getMessage());\r
-            }\r
+            intlogger.warn("PROV0007 doDelete: " + e.getMessage(), e);\r
         }\r
         return rv;\r
     }\r
@@ -551,8 +608,8 @@ public class Subscription extends Syncable {
         if (feedid != os.feedid) {\r
             return false;\r
         }\r
-        if (groupid != os.groupid) //New field is added - Groups feature Rally:US708115 - 1610\r
-        {\r
+        if (groupid != os.groupid) {\r
+            //New field is added - Groups feature Rally:US708115 - 1610\r
             return false;\r
         }\r
         if (!delivery.equals(os.delivery)) {\r
@@ -561,19 +618,27 @@ public class Subscription extends Syncable {
         if (metadataOnly != os.metadataOnly) {\r
             return false;\r
         }\r
+        if (followRedirect != os.followRedirect) {\r
+            return false;\r
+        }\r
         if (!subscriber.equals(os.subscriber)) {\r
             return false;\r
         }\r
         if (!links.equals(os.links)) {\r
             return false;\r
         }\r
-        return suspended == os.suspended;\r
+        if (suspended != os.suspended) {\r
+            return false;\r
+        }\r
+        if (!aafInstance.equals(os.aafInstance)) {\r
+            return false;\r
+        }\r
+        return true;\r
     }\r
 \r
     @Override\r
     public int hashCode() {\r
-        return Objects.hash(subid, feedid, groupid, delivery, metadataOnly, subscriber, links, suspended, lastMod,\r
-                createdDate);\r
+        return super.hashCode();\r
     }\r
 \r
     @Override\r