remove unused/dead code

Author: aoberoi

src/main/java/com/opentok/Archive.java

@@ -137,25 +137,6 @@ public Status getStatus() {
     public String getUrl() {
         return url;
     }
-    
-//    public Archive stop() throws OpenTokException {
-//        HashMap<String, String> headers = new HashMap<String, String>();
-//        headers.put("content-type", "application/json");
-//        String archive = HttpClient.makePostRequest("/v2/partner/" + this.partnerId + "/archive/" + id, headers, null,
-//                "{ \"action\" : \"stop\"  }");
-//        ObjectMapper mapper = new ObjectMapper();
-//        try {
-//            Archive updatedArchive =  mapper.readValue(archive, Archive.class);
-//            this.status = updatedArchive.status;
-//            return this;
-//        } catch (Exception e) {
-//            throw new RequestException(500, "Exception mapping json: " + e.getMessage());
-//        }
-//    }
-    
-//    public void delete() throws OpenTokException {
-//        HttpClient.makeDeleteRequest("/v2/partner/" + partnerId + "/archive/" + id);
-//    }
 
     @Override
     public String toString() {

src/main/java/com/opentok/OpenTok.java

@@ -351,6 +351,8 @@ public List<Archive> listArchives(int offset, int count) throws OpenTokException
             JsonParser jp = mapper.getFactory().createParser(archive);
             return mapper.readValue(mapper.treeAsTokens(mapper.readTree(jp).get("items")),
                     TypeFactory.defaultInstance().constructCollectionType(ArrayList.class, Archive.class));
+
+        // if we only wanted Java 7 and above, we could DRY this into one catch clause
         } catch (JsonMappingException e) {
             throw new RequestException("Exception mapping json: " + e.getMessage());
         } catch (JsonParseException e) {

src/main/java/com/opentok/exception/SessionNotFoundException.java

@@ -19,14 +19,11 @@
 public class HttpClient extends AsyncHttpClient {
 
     private final String apiUrl;
-    // NOTE: do we even need these since we now have the PartnerAuthRequestFilter?
     private final int apiKey;
-    private final String apiSecret;
 
     private HttpClient(Builder builder) {
         super(builder.config);
         this.apiKey = builder.apiKey;
-        this.apiSecret = builder.apiSecret;
         this.apiUrl = builder.apiUrl;
     }
 
@@ -238,76 +235,6 @@ public String deleteArchive(String archiveId) {
         return responseString;
     }
 
-//    protected static String makeDeleteRequest(String resource) throws OpenTokException {
-//        BoundRequestBuilder get = client.prepareDelete(apiUrl + resource);
-//        addCommonHeaders(get);
-//
-//        try {
-//            Response result = get.execute().get();
-//            if (result.getStatusCode() < 200 || result.getStatusCode() > 299) {
-//                throw new RequestException(result.getStatusCode(), "Error response: message: "
-//                        + result.getStatusText());
-//            }
-//            return result.getResponseBody();
-//        } catch (Exception e) {
-//            throw new RequestException(500, e.getMessage());
-//        }
-//    }
-//
-//    protected static String makeGetRequest(String resource) throws OpenTokException {
-//        BoundRequestBuilder get = client.prepareGet(apiUrl + resource);
-//        addCommonHeaders(get);
-//
-//        try {
-//            Response result = get.execute().get();
-//            if (result.getStatusCode() < 200 || result.getStatusCode() > 299) {
-//                throw new RequestException(result.getStatusCode(), "Error response: message: "
-//                        + result.getStatusText());
-//            }
-//            return result.getResponseBody();
-//        } catch (Exception e) {
-//            throw new RequestException(500, e.getMessage());
-//        }
-//    }
-//
-//    protected static String makePostRequest(String resource, Map<String, String> headers, Map<String, String> params,
-//            String postData) throws OpenTokException {
-//        BoundRequestBuilder post = client.preparePost(apiUrl + resource);
-//        if (params != null) {
-//            for (Entry<String, String> pair : params.entrySet()) {
-//                post.addParameter(pair.getKey(), pair.getValue());
-//            }
-//        }
-//
-//        if (headers != null) {
-//            for (Entry<String, String> pair : headers.entrySet()) {
-//                post.addHeader(pair.getKey(), pair.getValue());
-//            }
-//        }
-//
-//        addCommonHeaders(post);
-//
-//        if (postData != null) {
-//            post.setBody(postData);
-//        }
-//
-//        try {
-//            Response result = post.execute().get();
-//
-//            if (result.getStatusCode() < 200 || result.getStatusCode() > 299) {
-//                throw new RequestException(result.getStatusCode(), "Error response: message: " + result.getStatusText());
-//            }
-//
-//            return result.getResponseBody();
-//        } catch (Exception e) {
-//            throw new RequestException(500, e.getMessage());
-//        }
-//    }
-//
-//    private static void addCommonHeaders(BoundRequestBuilder get) {
-//        get.addHeader("X-TB-PARTNER-AUTH", String.format("%s:%s", apiKey, apiSecret));
-//    }
-
     public static class Builder {
         private final int apiKey;
         private final String apiSecret;