Network connections are compatible with lower versions

Author: shuzijun

src/main/java/com/shuzijun/leetcode/plugin/actions/toolbar/PickAction.java

@@ -1,6 +1,7 @@
 package com.shuzijun.leetcode.plugin.actions.toolbar;
 
 import com.intellij.openapi.actionSystem.AnActionEvent;
+import com.intellij.openapi.application.ApplicationManager;
 import com.intellij.ui.components.JBScrollPane;
 import com.shuzijun.leetcode.plugin.actions.AbstractAction;
 import com.shuzijun.leetcode.plugin.manager.ViewManager;
@@ -21,6 +22,8 @@ public void actionPerformed(AnActionEvent anActionEvent, Config config) {
             return;
         }
         JBScrollPane scrollPane = WindowFactory.getDataContext(anActionEvent.getProject()).getData(DataKeys.LEETCODE_PROJECTS_SCROLL);
-        ViewManager.pick(tree, scrollPane);
+        ApplicationManager.getApplication().invokeAndWait(() -> {
+            ViewManager.pick(tree, scrollPane);
+        });
     }
 }

src/main/java/com/shuzijun/leetcode/plugin/manager/ViewManager.java

@@ -184,8 +184,11 @@ public static void pick(JTree tree, JBScrollPane scrollPane) {
         }
         int i = (int) (Math.random() * node.getChildCount());
         DefaultMutableTreeNode select = (DefaultMutableTreeNode) node.getChildAt(i);
-        tree.setSelectionPath(new TreePath(select.getPath()));
-        Point point = new Point(0, i < 3 ? 0 : (i - 3) * tree.getRowHeight());
+
+        TreePath toShowPath = new TreePath(select.getPath());
+        tree.setSelectionPath(toShowPath);
+        Rectangle bounds = tree.getPathBounds(toShowPath);
+        Point point = new Point(0, (int) bounds.getY());
         JViewport viewport = scrollPane.getViewport();
         viewport.setViewPosition(point);
         return;

src/main/java/com/shuzijun/leetcode/plugin/utils/HttpRequestUtils.java

@@ -1,6 +1,6 @@
 package com.shuzijun.leetcode.plugin.utils;
 
-import com.intellij.util.io.HttpRequests;
+import com.shuzijun.leetcode.plugin.utils.io.HttpRequests;
 import org.apache.commons.lang3.StringUtils;
 import org.apache.http.HttpHeaders;
 import org.jetbrains.annotations.NotNull;
@@ -15,40 +15,40 @@
  */
 public class HttpRequestUtils {
 
-    private static CookieManager cookieManager = new CookieManager();
+    private static final CookieManager cookieManager = new CookieManager();
 
     static {
         CookieHandler.setDefault(cookieManager);
     }
 
     public static HttpResponse executeGet(HttpRequest httpRequest) {
 
+        HttpResponse httpResponse = new HttpResponse();
         try {
-            return HttpRequests.request(httpRequest.getUrl())
+            HttpRequests.request(httpRequest.getUrl())
                     .throwStatusCodeException(false)
                     .tuner(new HttpRequestTuner(httpRequest))
-                    .connect(new HttpResponseProcessor(httpRequest));
+                    .connect(new HttpResponseProcessor(httpRequest, httpResponse));
+
         } catch (IOException e) {
             LogUtils.LOG.error("HttpRequestUtils request error:", e);
-            HttpResponse httpResponse = new HttpResponse();
             httpResponse.setStatusCode(-1);
-            return httpResponse;
         }
+        return httpResponse;
     }
 
     public static HttpResponse executePost(HttpRequest httpRequest) {
-
+        HttpResponse httpResponse = new HttpResponse();
         try {
-            return HttpRequests.post(httpRequest.getUrl(), httpRequest.getContentType())
+            HttpRequests.post(httpRequest.getUrl(), httpRequest.getContentType())
                     .throwStatusCodeException(false)
                     .tuner(new HttpRequestTuner(httpRequest))
-                    .connect(new HttpResponseProcessor(httpRequest));
+                    .connect(new HttpResponseProcessor(httpRequest, httpResponse));
         } catch (IOException e) {
             LogUtils.LOG.error("HttpRequestUtils request error:", e);
-            HttpResponse httpResponse = new HttpResponse();
             httpResponse.setStatusCode(-1);
-            return httpResponse;
         }
+        return httpResponse;
     }
 
     public static String getToken() {
@@ -67,9 +67,6 @@ public static String getToken() {
     public static boolean isLogin() {
         HttpRequest request = HttpRequest.get(URLUtils.getLeetcodePoints());
         HttpResponse response = executeGet(request);
-        if (response == null) {
-            return Boolean.FALSE;
-        }
         if (response.getStatusCode() == 200) {
             return Boolean.TRUE;
         }
@@ -88,6 +85,18 @@ public static void resetHttpclient() {
         cookieManager.getCookieStore().removeAll();
     }
 
+
+    private static void defaultHeader(HttpRequest httpRequest) {
+        Map<String, String> header = httpRequest.getHeader();
+        header.putIfAbsent(HttpHeaders.USER_AGENT, "Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.119 Safari/537.36");
+        header.putIfAbsent(HttpHeaders.ACCEPT, "*/*");
+        //header.putIfAbsent(HttpHeaders.ACCEPT_ENCODING, "gzip, deflate, br");
+        header.putIfAbsent(HttpHeaders.ACCEPT_LANGUAGE, "zh-CN,zh;q=0.9");
+        header.putIfAbsent("origin", URLUtils.getLeetcodeUrl());
+        //header.putIfAbsent(":authority", URLUtils.getLeetcodeHost());
+        //header.putIfAbsent(":scheme", "https");
+    }
+
     private static class HttpRequestTuner implements HttpRequests.ConnectionTuner {
 
         private HttpRequest httpRequest;
@@ -108,48 +117,39 @@ public void tune(@NotNull URLConnection urlConnection) throws IOException {
             defaultHeader(httpRequest);
             httpRequest.getHeader().forEach((k, v) -> urlConnection.addRequestProperty(k, v));
         }
-
-        private static void defaultHeader(HttpRequest httpRequest) {
-            Map<String, String> header = httpRequest.getHeader();
-            header.putIfAbsent(HttpHeaders.USER_AGENT, "Mozilla/5.0 (Windows NT 6.1; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.119 Safari/537.36");
-            header.putIfAbsent(HttpHeaders.ACCEPT, "*/*");
-            //header.putIfAbsent(HttpHeaders.ACCEPT_ENCODING, "gzip, deflate, br");
-            header.putIfAbsent(HttpHeaders.ACCEPT_LANGUAGE, "zh-CN,zh;q=0.9");
-            header.putIfAbsent("origin", URLUtils.getLeetcodeUrl());
-            //header.putIfAbsent(":authority", URLUtils.getLeetcodeHost());
-            //header.putIfAbsent(":scheme", "https");
-        }
-
     }
 
 
     private static class HttpResponseProcessor implements HttpRequests.RequestProcessor<HttpResponse> {
 
-        private HttpRequest httpRequest;
+        private final HttpRequest httpRequest;
+        private final HttpResponse httpResponse;
 
-        public HttpResponseProcessor(HttpRequest httpRequest) {
+        public HttpResponseProcessor(HttpRequest httpRequest, HttpResponse httpResponse) {
             this.httpRequest = httpRequest;
+            this.httpResponse = httpResponse;
         }
 
         @Override
         public HttpResponse process(@NotNull HttpRequests.Request request) throws IOException {
+
             if (StringUtils.isNoneBlank(httpRequest.getBody())) {
                 request.write(httpRequest.getBody());
             }
-            HttpResponse httpResponse = new HttpResponse();
 
             URLConnection urlConnection = request.getConnection();
 
             if (!(urlConnection instanceof HttpURLConnection)) {
                 httpResponse.setStatusCode(-1);
+                return httpResponse;
             } else {
                 httpResponse.setStatusCode(((HttpURLConnection) urlConnection).getResponseCode());
             }
+
             try {
                 httpResponse.setBody(request.readString());
             } catch (IOException ignore) {
             }
-
             return httpResponse;
         }
     }