Upgrade jetty version (#17)

sofastack · May 10, 2019 · d096114 · d096114
1 parent fb83614
commit d096114
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/pom.xml b/pom.xml
@@ -78,7 +78,7 @@
         <jraft.version>1.2.4</jraft.version>
         <metrics.version>4.0.2</metrics.version>
         <commons-io.version>2.4</commons-io.version>
-        <jetty.version>9.4.12.v20180830</jetty.version>
+        <jetty.version>[9.4.17.v20190418,)</jetty.version>
         <main.user.dir>${user.dir}</main.user.dir>
         <argLine>-Dnetwork_interface_denylist=docker0</argLine>
     </properties>

diff --git a/...src/main/java/com/alipay/sofa/registry/server/session/resource/SessionDigestResource.java b/...src/main/java/com/alipay/sofa/registry/server/session/resource/SessionDigestResource.java
@@ -37,6 +37,7 @@
 import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashMap;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 
@@ -146,18 +147,17 @@ public Map<String, Object> getPushSwitch() {
     @Path("getDataInfoIdList")
     @Produces(MediaType.APPLICATION_JSON)
     public Collection<String> getDataInfoIdList() {
-        Collection<String> ret = new ArrayList<>();
+        Collection<String> ret = new HashSet<>();
         ret.addAll(sessionInterests.getInterestDataInfoIds());
         ret.addAll(sessionDataStore.getStoreDataInfoIds());
-        return sessionInterests.getInterestDataInfoIds();
+        return ret;
     }
 
     @GET
     @Path("checkSumDataInfoIdList")
     @Produces(MediaType.APPLICATION_JSON)
     public int checkSumDataInfoIdList() {
-        return sessionInterests.getInterestDataInfoIds().hashCode()
-               + sessionDataStore.getStoreDataInfoIds().hashCode();
+        return getDataInfoIdList().hashCode();
     }
 
     private void fillServerList(String type,

diff --git a/...c/test/java/com/alipay/sofa/registry/test/resource/session/SessionDigestResourceTest.java b/...c/test/java/com/alipay/sofa/registry/test/resource/session/SessionDigestResourceTest.java
@@ -183,6 +183,6 @@ public void testGetDataInfoIdList() {
     public void testCheckSumDataInfoIdList() {
         int result = sessionChannel.getWebTarget().path("digest/checkSumDataInfoIdList")
             .request(APPLICATION_JSON).get(int.class);
-        assertTrue(result > 0);
+        assertTrue(result != 0);
     }
 }