Merge pull request #96 from WeBankFinTech/release/1.4.2

Merge Release/1.4.2 to master

.ci/script/build-ci.sh

@@ -7,6 +7,8 @@ weid_config_tpl=${java_source_code_dir}/src/main/resources/weidentity.properties
 weid_config=${java_source_code_dir}/src/main/resources/weidentity.properties
 font=${java_source_code_dir}/src/main/resources/NotoSansCJKtc-Regular.ttf
 
+JAVA_OPTS='-Djdk.tls.namedGroups="secp256k1"'
+
 function modify_config()
 {
     echo "begin to modify sdk config..."
@@ -87,7 +89,7 @@ function deploy_contract()
         CLASSPATH=${CLASSPATH}:${jar_file}
     done
 
-    java -cp "$CLASSPATH" com.webank.weid.contract.deploy.DeployContract
+    java ${JAVA_OPTS} -cp "$CLASSPATH" com.webank.weid.contract.deploy.DeployContract
     echo "contract deployment done."
 }
 

.github/CONTRIBUTING.md

@@ -8,7 +8,7 @@
 
 ## 1. 发现 bug
 
-使用过程中遇到的任何问题，可以查看 [FAQ](../docs/zh_CN/docs/faq.md) ，或者给我们提 [issue](https://github.com/WeBankFinTech/WeIdentity/issues)。
+使用过程中遇到的任何问题，可以查看 [FAQ](https://github.com/WeBankFinTech/weid-doc/blob/master/docs/zh_CN/docs/faq.md)，或者给我们提 [issue](https://github.com/WeBankFinTech/WeIdentity/issues)。
 
 ## 2. 优化建议
 
@@ -24,16 +24,16 @@
 
 #### 分支管理
 
-代码类工程我们使用 [git-flow](https://nvie.com/posts/a-successful-git-branching-model/) 进行分支管理。包括这些工程： [weid-java-sdk](https://github.com/WeBankFinTech/weid-java-sdk)， [weid-contract](https://github.com/WeBankFinTech/weid-contract)。
-文档类工程我们使用 [github-flow](http://scottchacon.com/2011/08/31/github-flow.html) 进行分支管理。包括这些工程： [WeIdentity](https://github.com/WeBankFinTech/WeIdentity)。
+代码类工程我们使用 [git-flow](https://nvie.com/posts/a-successful-git-branching-model/) 进行分支管理。包括这些工程： [WeIdentity](https://github.com/WeBankFinTech/WeIdentity)，[weid-contract](https://github.com/WeBankFinTech/weid-contract)，[weid-build-tools](https://github.com/WeBankFinTech/weid-build-tools)，[weid-sample](https://github.com/WeBankFinTech/weid-sample)。
+文档类工程我们使用 [github-flow](http://scottchacon.com/2011/08/31/github-flow.html) 进行分支管理。包括这些工程：[weid-doc](https://github.com/WeBankFinTech/weid-doc)。
 
 #### 代码规范
 
-建议先浏览 [WeIdentity 代码规范](../docs/zh_CN/docs/styleguides/styleguides.md)。
+建议先浏览 [WeIdentity 代码规范](https://github.com/WeBankFinTech/weid-doc/blob/master/docs/zh_CN/docs/styleguides/styleguides.md)。
 
 #### 测试
 
-所有提交的代码必须通过单元测试，如果提交的代码目前的单元测试用例无法覆盖，可以修改单元测试代码从而覆盖用例，或者在 Pull Request 里面说明情况，然后由我们来补充单元测试用例。详见“[如何运行单元测试](../docs/zh_CN/docs/how-to-run-unit-test.md)”。
+所有提交的代码必须通过单元测试，如果提交的代码目前的单元测试用例无法覆盖，可以修改单元测试代码从而覆盖用例，或者在 Pull Request 里面说明情况，然后由我们来补充单元测试用例。详见“[如何运行单元测试](https://weidentity.readthedocs.io/zh_CN/latest/docs/how-to-run-unit-test.html)”。
 
 #### Commit Messages
 
@@ -55,7 +55,7 @@ data & logic contracts.
 
 ### 3.2 流程
 
-1. Fork 希望提交代码修改的 repository 。例如，如果想要修改 [weid-java-sdk](https://github.com/WeBankFinTech/weid-java-sdk) ，则 Fork 这个 repository。
+1. Fork 希望提交代码修改的 repository 。例如，如果想要修改 [WeIdentity](https://github.com/WeBankFinTech/WeIdentity) ，则 Fork 这个 repository。
 
 2. 创建新的分支，例如分支取名为 `feature/add-verfiy-logic` （分支名请使用 `feature/` 开头）。
 

CHANGELOG.md

@@ -1,3 +1,15 @@
+### V1.4.2 (2019-12-10)
+- Features:
+1. Allow WeID owners to remove unused public keys and authentications in WeID Document.
+2. Embedded CredentialPojo now supports change disclosure policy of inner CredentialPojo.
+3. WeIdentity now supports deployment on different FISCO-BCOS Groups (check docs for details).
+
+- Bugfixes:
+1. Fix various issues when serializing Embedded CredentialPojo.
+2. Bump jackson, jacoco and jmockit for security concerns.
+3. WeIdentity can now work with JDK1.8+ up to openjdk13.
+4. Fix broken Document URLs and simplify the document repository structure.
+
 ### V1.4.1 (2019-11-01)
 - Bugfixes:
 1. Fixed multiple potential leaks which might cause NPE.

build-tools/bin/run.sh

@@ -6,7 +6,9 @@ echo $source_code_dir
 cd $source_code_dir
 
 if [ -d dist/ ];then
-    rm -rf dist/
+    rm -rf dist/app
+    rm -rf dist/conf
+    rm -rf dist/*.jar
 fi
 
 gradle clean build -x checkMain -x checkTest -x spotbugsMain -x spotbugsTest -x test

build-tools/bin/setup.sh

@@ -8,6 +8,8 @@ app_xml_config=${java_source_code_dir}/src/main/resources/fisco.properties
 weid_config_tpl=${java_source_code_dir}/src/main/resources/weidentity.properties.tpl
 weid_config=${java_source_code_dir}/src/main/resources/weidentity.properties
 
+JAVA_OPTS='-Djdk.tls.namedGroups="secp256k1"'
+
 CLASSPATH=${java_source_code_dir}/dist/conf
 
 for jar_file in ${java_source_code_dir}/dist/lib/*.jar
@@ -145,7 +147,9 @@ function gradle_build_sdk()
 
     cd ${java_source_code_dir}/
     if [ -d dist/ ];then
-	rm -rf dist/
+        rm -rf dist/app
+        rm -rf dist/conf
+        rm -rf dist/*.jar
     fi
     gradle clean build -x checkMain -x checkTest -x spotbugsMain -x spotbugsTest -x test
     echo "compile java code done."
@@ -160,7 +164,7 @@ function deploy_contract()
 	CLASSPATH=${CLASSPATH}:${jar_file}
 	done
 
-    java -cp "$CLASSPATH" com.webank.weid.contract.deploy.DeployContract
+    java ${JAVA_OPTS} -cp "$CLASSPATH" com.webank.weid.contract.deploy.DeployContract
     echo "contract deployment done."
 }
 

build.gradle

@@ -36,7 +36,7 @@ if (!gradle.startParameter.isOffline()) {
 
 group 'com.webank'
 
-version = "1.4.1"
+version = "1.4.2"
 
 // Specify JDK version - may vary in different scenarios
 sourceCompatibility = 1.8
@@ -81,11 +81,11 @@ List apache_commons = [
 ]
 
 List jmockit = [
-        "org.jmockit:jmockit:1.9"
+        "org.jmockit:jmockit:1.47"
 ]
 
 List json = [
-        "com.fasterxml.jackson.core:jackson-databind:2.8.8.1",
+        "com.fasterxml.jackson.core:jackson-databind:2.10.0",
         "com.github.fge:json-schema-validator:2.2.6",
         "com.github.reinert:jjschema:1.16"
 ]
@@ -179,6 +179,11 @@ task javadocJar(type: Jar, dependsOn: javadoc) {
     from 'build/docs/javadoc'
 }
 
+test {
+   systemProperty "jdk.tls.namedGroups", "${jdkTlsNamedGroups}"
+   jvmArgs "-javaagent:${classpath.find { it.name.contains("jmockit") }.absolutePath}"
+}
+
 artifacts {
     archives jar
     archives sourcesJar
@@ -225,7 +230,11 @@ if (!gradle.startParameter.isOffline()) {
             html.enabled false
         }
     }
-
+
+	jacoco{
+        toolVersion = "0.8.5"
+    }
+
     check.dependsOn jacocoTestReport
 
     checkstyle {

check-info.sh

@@ -1,5 +1,6 @@
 #!/bin/bash
 
+JAVA_OPTS='-Djdk.tls.namedGroups="secp256k1"'
 classpathDir="./dist/conf"
 libDir="./dist/lib"
 set -- `getopt c:l: "$@"`
@@ -260,7 +261,7 @@ function check_node_version() {
 			then
 				echo "WARN: the current version of SDK does not support to check the node version, minimum version 1.4.0"
 			else
-				java -cp $libDir/*:$classpathDir/ com.webank.weid.app.AppCommand --checkversion test
+				java ${JAVA_OPTS} -cp $libDir/*:$classpathDir/ com.webank.weid.app.AppCommand --checkversion test
 			fi
 		fi
 	done
@@ -279,7 +280,7 @@ function check_node_version() {
 						echo "WARN: the current version of SDK does not support to check the node version, minimum version 1.4.0"
 					else
 						currentDir=`pwd`
-						java -cp $currentDir/dist/lib/*:$currentDir/dist/app/*:$currentDir/dist/conf/ com.webank.weid.app.AppCommand --checkversion test
+						java ${JAVA_OPTS} -cp $currentDir/dist/lib/*:$currentDir/dist/app/*:$currentDir/dist/conf/ com.webank.weid.app.AppCommand --checkversion test
 					fi
 				fi
 			done

config/checkstyle/webank_google_checks.xml

@@ -25,7 +25,7 @@
     </module>
 
     <module name="SuppressionFilter">
-        <property name="file" value="config/checkstyle/suppressions.xml"/>
+        <property name="file" value="${config_loc}/suppressions.xml"/>
         <property name="optional" value="false"/>
     </module>
     <module name="SuppressWarningsFilter"/>