Refactor Prov DB handling
[dmaap/datarouter.git] / datarouter-prov / src / main / java / org / onap / dmaap / datarouter / provisioning / beans / Subscription.java
index 1333b55..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.*;\r
-\r
-import org.apache.log4j.Logger;\r
+import java.util.ArrayList;\r
+import java.util.Collection;\r
+import java.util.Date;\r
+import java.util.List;\r
+import java.util.Properties;\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
-    private static Logger intlogger = Logger.getLogger("org.onap.dmaap.datarouter.provisioning.internal");\r
-    private static int next_subid = getMaxSubID() + 1;\r
+\r
+    private static final String SQLEXCEPTION = "SQLException: ";\r
+    private static final String SUBID_KEY = "subid";\r
+    private static final String SUBID_COL = "SUBID";\r
+    private static final String FEEDID_KEY = "feedid";\r
+    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 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
     private boolean suspended;\r
-    private Date last_mod;\r
-    private Date created_date;\r
+    private Date lastMod;\r
+    private Date createdDate;\r
+    private boolean privilegedSubscriber;\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
+        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
+            // The JSONObject is assumed to contain a vnd.dmaap-dr.subscription representation\r
+            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
+            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
+            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 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() > 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
+            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
+            intlogger.warn("Invalid JSON: " + e.getMessage(), e);\r
+            throw new InvalidObjectException("Invalid JSON: " + e.getMessage());\r
+        }\r
+    }\r
 \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\" 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
+            "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.size() > 0 ? list.get(0) : null;\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.size() > 0 ? list.get(0) : null;\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<Subscription>();\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
+        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
-            db.release(conn);\r
         } catch (SQLException e) {\r
-            e.printStackTrace();\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 {\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
+        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
-            db.release(conn);\r
         } catch (SQLException e) {\r
-            intlogger.info("getMaxSubID: " + e.getMessage());\r
-            e.printStackTrace();\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<String>();\r
-        String sql = "select SUBID from SUBSCRIPTIONS where FEEDID = " + feedid;\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
-                        int subid = rs.getInt("SUBID");\r
-                        list.add(URLUtilities.generateSubscriptionURL(subid));\r
-                    }\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
-            db.release(conn);\r
         } catch (SQLException e) {\r
-            e.printStackTrace();\r
+            intlogger.error(SQLEXCEPTION + e.getMessage(), e);\r
         }\r
         return list;\r
     }\r
@@ -153,102 +284,31 @@ public class Subscription extends Syncable {
      */\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
+        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
-            db.release(conn);\r
         } catch (SQLException e) {\r
-            intlogger.warn("PROV0008 countActiveSubscriptions: " + e.getMessage());\r
-            e.printStackTrace();\r
+            intlogger.warn("PROV0008 countActiveSubscriptions: " + e.getMessage(), e);\r
         }\r
         return count;\r
     }\r
 \r
-    public Subscription() {\r
-        this("", "", "");\r
-    }\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.subscriber = "";\r
-        this.links = new SubLinks();\r
-        this.suspended = false;\r
-        this.last_mod = new Date();\r
-        this.created_date = new Date();\r
-    }\r
-\r
-    public Subscription(ResultSet rs) throws SQLException {\r
-        this.subid = rs.getInt("SUBID");\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.subscriber = rs.getString("SUBSCRIBER");\r
-        this.links = new SubLinks(rs.getString("SELF_LINK"), URLUtilities.generateFeedURL(feedid), rs.getString("LOG_LINK"));\r
-        this.suspended = rs.getBoolean("SUSPENDED");\r
-        this.last_mod = rs.getDate("LAST_MOD");\r
-        this.created_date = rs.getDate("CREATED_DATE");\r
-    }\r
-\r
-    public Subscription(JSONObject jo) throws InvalidObjectException {\r
-        this("", "", "");\r
-        try {\r
-            // The JSONObject is assumed to contain a vnd.att-dr.subscription representation\r
-            this.subid = jo.optInt("subid", -1);\r
-            this.feedid = jo.optInt("feedid", -1);\r
-            this.groupid = jo.optInt("groupid", -1); //New field is added - Groups feature Rally:US708115 - 1610\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
-\r
-\r
-            //Data Router Subscriber HTTPS Relaxation feature USERSTORYID:US674047.\r
-            Properties p = (new DB()).getProperties();\r
-            if (p.get("org.onap.dmaap.datarouter.provserver.https.relaxation").toString().equals("false") && !jo.has("sync")) {\r
-                if (!url.startsWith("https://"))\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
-            if (user.length() > 20)\r
-                throw new InvalidObjectException("delivery user field is too long");\r
-            if (password.length() > 32)\r
-                throw new InvalidObjectException("delivery password field is too long");\r
-            this.delivery = new SubDelivery(url, user, password, use100);\r
-\r
-            this.metadataOnly = jo.getBoolean("metadataOnly");\r
-            this.suspended = jo.optBoolean("suspend", false);\r
-\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
-        }\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
@@ -262,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
@@ -270,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
@@ -295,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
@@ -303,14 +383,27 @@ public class Subscription extends Syncable {
         this.suspended = suspended;\r
     }\r
 \r
+    public boolean isPrivilegedSubscriber() {\r
+        return privilegedSubscriber;\r
+    }\r
+\r
+    public void setPrivilegedSubscriber(boolean privilegedSubscriber) {\r
+        this.privilegedSubscriber = privilegedSubscriber;\r
+    }\r
+\r
     public String getSubscriber() {\r
         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
+            if (subscriber.length() > 8) {\r
                 subscriber = subscriber.substring(0, 8);\r
+            }\r
             this.subscriber = subscriber;\r
         }\r
     }\r
@@ -319,61 +412,87 @@ 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
-        jo.put("subid", subid);\r
-        jo.put("feedid", feedid);\r
-        jo.put("groupid", groupid); //New field is added - Groups feature Rally:US708115 - 1610\r
+        jo.put(SUBID_KEY, subid);\r
+        jo.put(FEEDID_KEY, feedid);\r
+        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", last_mod.getTime());\r
-        jo.put("created_date", created_date.getTime());\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
+    /**\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
-        jo.remove("subid");\r
-        jo.remove("feedid");\r
-        jo.remove("last_mod");\r
+        if (hidepasswords) {\r
+            jo.remove(SUBID_KEY);    // we no longer hide passwords, however we do hide these\r
+            jo.remove(FEEDID_KEY);\r
+            jo.remove(LAST_MOD_KEY);\r
+            jo.remove(CREATED_DATE);\r
+        }\r
         return jo;\r
     }\r
 \r
-    public JSONObject asJSONObject(boolean hidepasswords) {\r
+    /**\r
+     * Method to remove some attributes from JSON.\r
+     * @ JSONObject\r
+     */\r
+    public JSONObject asLimitedJSONObject() {\r
         JSONObject jo = asJSONObject();\r
-        if (hidepasswords) {\r
-            jo.remove("subid");    // we no longer hide passwords, however we do hide these\r
-            jo.remove("feedid");\r
-            jo.remove("last_mod");\r
-            jo.remove("created_date");\r
-        }\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
             if (subid == -1) {\r
                 // No feed ID assigned yet, so assign the next available one\r
-                setSubid(next_subid++);\r
+                setSubid(nextSubid++);\r
             }\r
             // In case we insert a feed from synchronization\r
-            if (subid > next_subid)\r
-                next_subid = subid + 1;\r
+            if (subid > nextSubid) {\r
+                nextSubid = subid + 1;\r
+            }\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) values (?, ?, ?, ?, ?, ?, ?, ?, ?, ?)";\r
-            ps = c.prepareStatement(sql, new String[]{"SUBID"});\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
@@ -384,11 +503,15 @@ public class Subscription extends Syncable {
             ps.setString(8, getSubscriber());\r
             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
@@ -396,27 +519,26 @@ public class Subscription extends Syncable {
             ps.close();\r
         } catch (SQLException e) {\r
             rv = false;\r
-            intlogger.warn("PROV0005 doInsert: " + e.getMessage());\r
-            e.printStackTrace();\r
+            intlogger.warn("PROV0005 doInsert: " + e.getMessage(), e);\r
         } finally {\r
             try {\r
-                if(ps!=null) {\r
+                if (ps != null) {\r
                     ps.close();\r
                 }\r
             } catch (SQLException e) {\r
-                e.printStackTrace();\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 = ? 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
@@ -424,81 +546,47 @@ public class Subscription extends Syncable {
             ps.setInt(5, isMetadataOnly() ? 1 : 0);\r
             ps.setInt(6, suspended ? 1 : 0);\r
             ps.setInt(7, groupid); //New field is added - Groups feature Rally:US708115 - 1610\r
-            ps.setInt(8, subid);\r
+            ps.setInt(8, privilegedSubscriber ? 1 : 0);\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
-            e.printStackTrace();\r
-        } finally {\r
-            try {\r
-                if(ps!=null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                e.printStackTrace();\r
-            }\r
+            intlogger.warn("PROV0006 doUpdate: " + e.getMessage(), e);\r
         }\r
         return rv;\r
     }\r
 \r
 \r
     /**\r
-     * Rally US708115\r
-     * 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
-            e.printStackTrace();\r
-        } finally {\r
-            try {\r
-                if(ps!=null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                e.printStackTrace();\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
-            e.printStackTrace();\r
-        } finally {\r
-            try {\r
-                if(ps!=null) {\r
-                    ps.close();\r
-                }\r
-            } catch (SQLException e) {\r
-                e.printStackTrace();\r
-            }\r
+            intlogger.warn("PROV0007 doDelete: " + e.getMessage(), e);\r
         }\r
         return rv;\r
     }\r
@@ -510,31 +598,47 @@ public class Subscription extends Syncable {
 \r
     @Override\r
     public boolean equals(Object obj) {\r
-        if (!(obj instanceof Subscription))\r
+        if (!(obj instanceof Subscription)) {\r
             return false;\r
+        }\r
         Subscription os = (Subscription) obj;\r
-        if (subid != os.subid)\r
+        if (subid != os.subid) {\r
             return false;\r
-        if (feedid != os.feedid)\r
+        }\r
+        if (feedid != os.feedid) {\r
+            return false;\r
+        }\r
+        if (groupid != os.groupid) {\r
+            //New field is added - Groups feature Rally:US708115 - 1610\r
             return false;\r
-        if (groupid != os.groupid) //New field is added - Groups feature Rally:US708115 - 1610\r
+        }\r
+        if (!delivery.equals(os.delivery)) {\r
+            return false;\r
+        }\r
+        if (metadataOnly != os.metadataOnly) {\r
             return false;\r
-        if (!delivery.equals(os.delivery))\r
+        }\r
+        if (followRedirect != os.followRedirect) {\r
             return false;\r
-        if (metadataOnly != os.metadataOnly)\r
+        }\r
+        if (!subscriber.equals(os.subscriber)) {\r
             return false;\r
-        if (!subscriber.equals(os.subscriber))\r
+        }\r
+        if (!links.equals(os.links)) {\r
             return false;\r
-        if (!links.equals(os.links))\r
+        }\r
+        if (suspended != os.suspended) {\r
             return false;\r
-        if (suspended != os.suspended)\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, last_mod, created_date);\r
+        return super.hashCode();\r
     }\r
 \r
     @Override\r