[GitHub] [incubator-teaclave] uraj opened a new pull request #151: Exclude test code in normal build

2019-12-13 Thread GitBox
uraj opened a new pull request #151: Exclude test code in normal build URL: https://github.com/apache/incubator-teaclave/pull/151 ## Description Guard test-related code with cfg flags Fixes #127 ## Type of change (select applied and DELETE the others) - [ ] Bug f

[GitHub] [incubator-teaclave] mssun opened a new issue #152: Use TOML format to store enclave info

2019-12-13 Thread GitBox
mssun opened a new issue #152: Use TOML format to store enclave info URL: https://github.com/apache/incubator-teaclave/issues/152 Current enclave info is stored in the `enclave_info.txt` which is a customize format. I'm proposing to use TOML for enclave_info. Here is an example: `en

[GitHub] [incubator-teaclave] mssun commented on issue #152: Use TOML format to store enclave info

2019-12-13 Thread GitBox
mssun commented on issue #152: Use TOML format to store enclave info URL: https://github.com/apache/incubator-teaclave/issues/152#issuecomment-565626421 I updated the deserialization code so that it auto deserialize hex `String` to `Vec`. --

[GitHub] [incubator-teaclave] mssun merged pull request #151: Exclude test code in normal build

2019-12-13 Thread GitBox
mssun merged pull request #151: Exclude test code in normal build URL: https://github.com/apache/incubator-teaclave/pull/151 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [incubator-teaclave] mssun closed issue #127: Put test related code under #[cfg(test_mode)] gate

2019-12-13 Thread GitBox
mssun closed issue #127: Put test related code under #[cfg(test_mode)] gate URL: https://github.com/apache/incubator-teaclave/issues/127 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-teaclave] mssun opened a new issue #153: CMake init_submodules function doesn't track the status of submodule properly

2019-12-13 Thread GitBox
mssun opened a new issue #153: CMake init_submodules function doesn't track the status of submodule properly URL: https://github.com/apache/incubator-teaclave/issues/153 The `init_submodules` function in cmake doesn't track the status of submodule properly. https://github.com/apach

[GitHub] [incubator-teaclave] mssun opened a new pull request #154: [ci] Don't mount SGX device for simulation mode

2019-12-13 Thread GitBox
mssun opened a new pull request #154: [ci] Don't mount SGX device for simulation mode URL: https://github.com/apache/incubator-teaclave/pull/154 ## Description Don't mount SGX device for simulation mode This is an

[GitHub] [incubator-teaclave] mssun merged pull request #154: [ci] Don't mount SGX device for simulation mode

2019-12-13 Thread GitBox
mssun merged pull request #154: [ci] Don't mount SGX device for simulation mode URL: https://github.com/apache/incubator-teaclave/pull/154 This is an automated message from the Apache Git Service. To respond to the message, p

[GitHub] [incubator-teaclave] mssun opened a new pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
mssun opened a new pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155 ## Description This PR introduce TOML format enclave_info to replace txt. ## Type of change (select applied and DELETE the others) - [x]

[GitHub] [incubator-teaclave] mssun opened a new issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils

2019-12-13 Thread GitBox
mssun opened a new issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils URL: https://github.com/apache/incubator-teaclave/issues/156 Since these utility functions are used in many projects core, sdk, and cli, we can separate it into one single project.

[GitHub] [incubator-teaclave] mssun commented on issue #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
mssun commented on issue #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#issuecomment-565684814 This PR also fix #156. This is an automated message from the Apache Git Ser

[GitHub] [incubator-teaclave] mssun opened a new issue #157: Introduce `make lint` which combines `make format` and `make clippy`

2019-12-13 Thread GitBox
mssun opened a new issue #157: Introduce `make lint` which combines `make format` and `make clippy` URL: https://github.com/apache/incubator-teaclave/issues/157 This will make life easier. This is an automated message from th

[GitHub] [incubator-teaclave] mssun opened a new issue #158: Auditor check is not compulsory for users' client sdk/cli

2019-12-13 Thread GitBox
mssun opened a new issue #158: Auditor check is not compulsory for users' client sdk/cli URL: https://github.com/apache/incubator-teaclave/issues/158 Yes, users *should* check the auditors signatures before the connection. However, this can be done offline with a separate tools or integrat

[GitHub] [incubator-teaclave] mssun commented on issue #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
mssun commented on issue #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#issuecomment-565685288 I'm done with this PR. Please review. The latest CI result can be seen here: https://ci.mesalock-linux.org/mssun/incubator-mesatee/129

[GitHub] [incubator-teaclave] m4sterchain commented on issue #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
m4sterchain commented on issue #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#issuecomment-565685593 > I'm done with this PR. Please review. The latest CI result can be seen here: https://ci.mesalock-linux.org/mssun/incubator-mesatee/1

[GitHub] [incubator-teaclave] mssun opened a new issue #159: The `sgx_launch_check` function is no longer needed

2019-12-13 Thread GitBox
mssun opened a new issue #159: The `sgx_launch_check` function is no longer needed URL: https://github.com/apache/incubator-teaclave/issues/159 This legacy function can be removed because of the newly introduced `teaclave_config`. They do same thing: https://github.com/apach

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#discussion_r357901331 ## File path: mesatee_utils/teaclave_utils/src/lib.rs ## @@ -0,0 +1,146 @@ +#![cfg_attr(feature = "

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#discussion_r357900744 ## File path: cmake/scripts/gen_enclave_info_toml.py ## @@ -0,0 +1,20 @@ +import sys + +def find_he

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
m4sterchain commented on a change in pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#discussion_r357900744 ## File path: cmake/scripts/gen_enclave_info_toml.py ## @@ -0,0 +1,20 @@ +import sys + +def find_he

[GitHub] [incubator-teaclave] mssun commented on a change in pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
mssun commented on a change in pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155#discussion_r357901980 ## File path: mesatee_utils/teaclave_utils/src/lib.rs ## @@ -0,0 +1,146 @@ +#![cfg_attr(feature = "mesalo

[GitHub] [incubator-teaclave] mssun closed issue #152: Use TOML format to store enclave info

2019-12-13 Thread GitBox
mssun closed issue #152: Use TOML format to store enclave info URL: https://github.com/apache/incubator-teaclave/issues/152 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-teaclave] mssun merged pull request #155: Use enclave_info.toml instead of txt

2019-12-13 Thread GitBox
mssun merged pull request #155: Use enclave_info.toml instead of txt URL: https://github.com/apache/incubator-teaclave/pull/155 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-teaclave] mssun opened a new pull request #160: Rename mesatee_utils to teaclave_common

2019-12-14 Thread GitBox
mssun opened a new pull request #160: Rename mesatee_utils to teaclave_common URL: https://github.com/apache/incubator-teaclave/pull/160 ## Description Rename `/mesatee_utils` to `/teaclave_common` and move `/mesatee_utils/teaclave_utils` to `/teaclave_utils` Fixes # (issue)

[GitHub] [incubator-teaclave] mssun merged pull request #160: Rename mesatee_utils to teaclave_common

2019-12-14 Thread GitBox
mssun merged pull request #160: Rename mesatee_utils to teaclave_common URL: https://github.com/apache/incubator-teaclave/pull/160 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-teaclave] mssun closed issue #147: Rename mesatee_utils to mesatee_common

2019-12-14 Thread GitBox
mssun closed issue #147: Rename mesatee_utils to mesatee_common URL: https://github.com/apache/incubator-teaclave/issues/147 This is an automated message from the Apache Git Service. To respond to the message, please log on t

[GitHub] [incubator-teaclave] mssun opened a new issue #161: Gracefully exit from SGX trusted lib

2019-12-14 Thread GitBox
mssun opened a new issue #161: Gracefully exit from SGX trusted lib URL: https://github.com/apache/incubator-teaclave/issues/161 For example, in teaclave_config, we `expect()` the `IAS_KEY` and `IAS_SPID`. This will cause panic when env vars are not found. Can we gracefully exit the progra

[GitHub] [incubator-teaclave] mssun opened a new pull request #162: [core] Remove unneccessary sgx_launch_check function

2019-12-14 Thread GitBox
mssun opened a new pull request #162: [core] Remove unneccessary sgx_launch_check function URL: https://github.com/apache/incubator-teaclave/pull/162 ## Description The `sgx_launch_check` function is no longer needed. Fixes #159 ## Type of change (select applied and DE

[GitHub] [incubator-teaclave] mssun closed issue #159: The `sgx_launch_check` function is no longer needed

2019-12-14 Thread GitBox
mssun closed issue #159: The `sgx_launch_check` function is no longer needed URL: https://github.com/apache/incubator-teaclave/issues/159 This is an automated message from the Apache Git Service. To respond to the message, pl

[GitHub] [incubator-teaclave] mssun merged pull request #162: [core] Remove unneccessary sgx_launch_check function

2019-12-14 Thread GitBox
mssun merged pull request #162: [core] Remove unneccessary sgx_launch_check function URL: https://github.com/apache/incubator-teaclave/pull/162 This is an automated message from the Apache Git Service. To respond to the mess

[GitHub] [incubator-teaclave] mssun commented on issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils

2019-12-14 Thread GitBox
mssun commented on issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils URL: https://github.com/apache/incubator-teaclave/issues/156#issuecomment-565759171 Resolved by #155. This is an automated message from

[GitHub] [incubator-teaclave] mssun closed issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils

2019-12-14 Thread GitBox
mssun closed issue #156: Separate mesatee_core/src/rpc/sgx/utils to mesatee_utils URL: https://github.com/apache/incubator-teaclave/issues/156 This is an automated message from the Apache Git Service. To respond to the messa

[GitHub] [incubator-teaclave] mssun opened a new pull request #163: Polish measurement and signature verification

2019-12-14 Thread GitBox
mssun opened a new pull request #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163 ## Description Polish measurement and signature verification. Separate into two functions: `verify_enclave_info` and `load_enclave_info`. sd

[GitHub] [incubator-teaclave] mssun opened a new pull request #164: Rename to Apache Teaclave (incubating) and add NOTICE file

2019-12-14 Thread GitBox
mssun opened a new pull request #164: Rename to Apache Teaclave (incubating) and add NOTICE file URL: https://github.com/apache/incubator-teaclave/pull/164 ## Description Update DISCLAIMER and add NOTICE in the Apache way [1] [2]. - [1]: https://incubator.apache.org/guides/bra

[GitHub] [incubator-teaclave] mssun opened a new pull request #165: Remove .editor_config

2019-12-14 Thread GitBox
mssun opened a new pull request #165: Remove .editor_config URL: https://github.com/apache/incubator-teaclave/pull/165 ## Description `.editor_config` is not needed. We can enforce code style by linting pipeline in CI. Also, the config in the file is not applied to current code. So,

[GitHub] [incubator-teaclave] mssun opened a new pull request #166: Add CONTRIBUTORS.md

2019-12-14 Thread GitBox
mssun opened a new pull request #166: Add CONTRIBUTORS.md URL: https://github.com/apache/incubator-teaclave/pull/166 ## Description Rewrite `MAINTAINERS.md` to `CONTRIBUTORS.md`. Make this file more informative. Th

[GitHub] [incubator-teaclave] mssun opened a new pull request #167: Add .dockerignore

2019-12-14 Thread GitBox
mssun opened a new pull request #167: Add .dockerignore URL: https://github.com/apache/incubator-teaclave/pull/167 ## Description Add `.dockerignore` file to avoid adding unnecessary files to building context which slows down the building process. - https://docs.docker.com/eng

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #163: Polish measurement and signature verification

2019-12-15 Thread GitBox
m4sterchain commented on a change in pull request #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163#discussion_r357993810 ## File path: keys/auditors/README.md ## @@ -16,7 +16,7 @@ sample entry looks like: ```

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #163: Polish measurement and signature verification

2019-12-15 Thread GitBox
m4sterchain commented on a change in pull request #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163#discussion_r357994253 ## File path: teaclave_utils/src/lib.rs ## @@ -73,16 +72,16 @@ pub fn percent_decode(orig:

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #163: Polish measurement and signature verification

2019-12-15 Thread GitBox
m4sterchain commented on a change in pull request #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163#discussion_r357995614 ## File path: mesatee_core/src/rpc/sgx/mod.rs ## @@ -60,40 +60,47 @@ pub fn prelude() {

[GitHub] [incubator-teaclave] mssun commented on issue #163: Polish measurement and signature verification

2019-12-15 Thread GitBox
mssun commented on issue #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163#issuecomment-565835876 Updated. This is an automated message from the Apache Git Service.

[GitHub] [incubator-teaclave] mssun merged pull request #165: Remove .editor_config

2019-12-15 Thread GitBox
mssun merged pull request #165: Remove .editor_config URL: https://github.com/apache/incubator-teaclave/pull/165 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-teaclave] mssun merged pull request #166: Add CONTRIBUTORS.md

2019-12-15 Thread GitBox
mssun merged pull request #166: Add CONTRIBUTORS.md URL: https://github.com/apache/incubator-teaclave/pull/166 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and u

[GitHub] [incubator-teaclave] mssun merged pull request #167: Add .dockerignore

2019-12-15 Thread GitBox
mssun merged pull request #167: Add .dockerignore URL: https://github.com/apache/incubator-teaclave/pull/167 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use

[GitHub] [incubator-teaclave] mssun merged pull request #164: Rename to Apache Teaclave (incubating) and add NOTICE file

2019-12-15 Thread GitBox
mssun merged pull request #164: Rename to Apache Teaclave (incubating) and add NOTICE file URL: https://github.com/apache/incubator-teaclave/pull/164 This is an automated message from the Apache Git Service. To respond to th

[GitHub] [incubator-teaclave] mssun merged pull request #163: Polish measurement and signature verification

2019-12-15 Thread GitBox
mssun merged pull request #163: Polish measurement and signature verification URL: https://github.com/apache/incubator-teaclave/pull/163 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-teaclave] mssun opened a new pull request #168: [cli] Rewrite command line tool

2019-12-15 Thread GitBox
mssun opened a new pull request #168: [cli] Rewrite command line tool URL: https://github.com/apache/incubator-teaclave/pull/168 ## Description Rewrite command line tool. Separate audit and connect as two subcommands. Fixes #158 ## Type of change (select applied and DELE

[GitHub] [incubator-teaclave-sgx-sdk] 60ke opened a new issue #183: 关于远程证实

2019-12-16 Thread GitBox
60ke opened a new issue #183: 关于远程证实 URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/183 During the IAS registration, one always generates two files: client.crt and client.key. 在intel注册完账号,只看到SPID和密匙,没见到client.crt client.key. ---

[GitHub] [incubator-teaclave] mssun commented on issue #168: [cli] Rewrite command line tool

2019-12-16 Thread GitBox
mssun commented on issue #168: [cli] Rewrite command line tool URL: https://github.com/apache/incubator-teaclave/pull/168#issuecomment-566169764 Yes, sure. This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-teaclave] mssun merged pull request #168: [cli] Rewrite command line tool

2019-12-16 Thread GitBox
mssun merged pull request #168: [cli] Rewrite command line tool URL: https://github.com/apache/incubator-teaclave/pull/168 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-teaclave] mssun closed issue #158: Auditor signatures check can be seperated from users' client sdk/cli

2019-12-16 Thread GitBox
mssun closed issue #158: Auditor signatures check can be seperated from users' client sdk/cli URL: https://github.com/apache/incubator-teaclave/issues/158 This is an automated message from the Apache Git Service. To respond

[GitHub] [incubator-teaclave] mssun opened a new issue #169: Remove dependency `quicli` in `teaclave_cli`

2019-12-16 Thread GitBox
mssun opened a new issue #169: Remove dependency `quicli` in `teaclave_cli` URL: https://github.com/apache/incubator-teaclave/issues/169 The only thing we need is `type CliResult = Result<(), ExitFailure>` in the `quicli::prelude`. I think we can remove this dependency. ---

[GitHub] [incubator-teaclave] mssun opened a new pull request #170: [cli] Remove unneccessary dependency

2019-12-16 Thread GitBox
mssun opened a new pull request #170: [cli] Remove unneccessary dependency URL: https://github.com/apache/incubator-teaclave/pull/170 ## Description `quicli` is not useful in `teaclave_cli` right now. Fixes #169 ## Type of change (select applied and DELETE the others)

[GitHub] [incubator-teaclave-sgx-sdk] dingelish commented on issue #183: 关于远程证实

2019-12-16 Thread GitBox
dingelish commented on issue #183: 关于远程证实 URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/183#issuecomment-566195525 您好,现在已经不需要crt/key了,因为IAS已经切换到了v5 API。在之前的IAS spec中是需要crt/key的,v5已经抛弃了这个接口。 对于不可信组件/可信组件之间的远程证实,请参考ue-ra例子。 对于可信组件之间的互相远程证实,请参考mutual-ra例子。 老

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #170: [cli] Remove unneccessary dependency

2019-12-16 Thread GitBox
m4sterchain commented on a change in pull request #170: [cli] Remove unneccessary dependency URL: https://github.com/apache/incubator-teaclave/pull/170#discussion_r358409574 ## File path: teaclave_cli/Cargo.toml ## @@ -8,7 +8,6 @@ edition = "2018" [dependencies] mesate

[GitHub] [incubator-teaclave] mssun commented on a change in pull request #170: [cli] Remove unneccessary dependency

2019-12-16 Thread GitBox
mssun commented on a change in pull request #170: [cli] Remove unneccessary dependency URL: https://github.com/apache/incubator-teaclave/pull/170#discussion_r358410672 ## File path: teaclave_cli/Cargo.toml ## @@ -8,7 +8,6 @@ edition = "2018" [dependencies] mesatee_core

[GitHub] [incubator-teaclave] m4sterchain commented on a change in pull request #170: [cli] Remove unneccessary dependency

2019-12-16 Thread GitBox
m4sterchain commented on a change in pull request #170: [cli] Remove unneccessary dependency URL: https://github.com/apache/incubator-teaclave/pull/170#discussion_r358411999 ## File path: teaclave_cli/Cargo.toml ## @@ -8,7 +8,6 @@ edition = "2018" [dependencies] mesate

[GitHub] [incubator-teaclave] mssun closed issue #169: Remove dependency `quicli` in `teaclave_cli`

2019-12-16 Thread GitBox
mssun closed issue #169: Remove dependency `quicli` in `teaclave_cli` URL: https://github.com/apache/incubator-teaclave/issues/169 This is an automated message from the Apache Git Service. To respond to the message, please lo

[GitHub] [incubator-teaclave] mssun merged pull request #170: [cli] Remove unneccessary dependency

2019-12-16 Thread GitBox
mssun merged pull request #170: [cli] Remove unneccessary dependency URL: https://github.com/apache/incubator-teaclave/pull/170 This is an automated message from the Apache Git Service. To respond to the message, please log o

[GitHub] [incubator-teaclave] mssun opened a new pull request #171: [binder] Polish binder

2019-12-16 Thread GitBox
mssun opened a new pull request #171: [binder] Polish binder URL: https://github.com/apache/incubator-teaclave/pull/171 ## Description - redefine ocalls - remove useless launch token code ## Type of change (select applied and DELETE the others) - [x] Code cleanup or j

[GitHub] [incubator-teaclave] mssun merged pull request #171: [binder] Polish binder

2019-12-16 Thread GitBox
mssun merged pull request #171: [binder] Polish binder URL: https://github.com/apache/incubator-teaclave/pull/171 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub an

[GitHub] [incubator-teaclave] mssun opened a new pull request #172: [config] Make runtime config optional and handle none config at enclave init

2019-12-17 Thread GitBox
mssun opened a new pull request #172: [config] Make runtime config optional and handle none config at enclave init URL: https://github.com/apache/incubator-teaclave/pull/172 ## Description Make runtime config optional and handle none config at enclave init Fixes #161 ##

[GitHub] [incubator-teaclave] mssun opened a new issue #173: `kms`'s test data is not gated by test_mode

2019-12-17 Thread GitBox
mssun opened a new issue #173: `kms`'s test data is not gated by test_mode URL: https://github.com/apache/incubator-teaclave/issues/173 https://github.com/apache/incubator-teaclave/blob/6f78e92d4a6a7e43c8275194a91fd0834dc1e114/mesatee_services/kms/sgx_trusted_lib/src/kms.rs#L38-L44 ---

[GitHub] [incubator-teaclave] mssun opened a new pull request #174: Remove customized CoC refer to using ASF CoC

2019-12-17 Thread GitBox
mssun opened a new pull request #174: Remove customized CoC refer to using ASF CoC URL: https://github.com/apache/incubator-teaclave/pull/174 ## Description ASF has CoC, no need to include ours: https://www.apache.org/foundation/policies/conduct

[GitHub] [incubator-teaclave] mssun commented on issue #172: [config] Make runtime config optional and handle none config at enclave init

2019-12-17 Thread GitBox
mssun commented on issue #172: [config] Make runtime config optional and handle none config at enclave init URL: https://github.com/apache/incubator-teaclave/pull/172#issuecomment-566647946 I don’t think helper functions can help. Getters and setters are not very common in Rust. It’s mo

[GitHub] [incubator-teaclave] mssun merged pull request #174: Remove customized CoC refer to using ASF CoC

2019-12-17 Thread GitBox
mssun merged pull request #174: Remove customized CoC refer to using ASF CoC URL: https://github.com/apache/incubator-teaclave/pull/174 This is an automated message from the Apache Git Service. To respond to the message, plea

[GitHub] [incubator-teaclave] mssun closed issue #161: Gracefully exit from SGX trusted lib

2019-12-17 Thread GitBox
mssun closed issue #161: Gracefully exit from SGX trusted lib URL: https://github.com/apache/incubator-teaclave/issues/161 This is an automated message from the Apache Git Service. To respond to the message, please log on to

[GitHub] [incubator-teaclave] mssun merged pull request #172: [config] Make runtime config optional and handle none config at enclave init

2019-12-17 Thread GitBox
mssun merged pull request #172: [config] Make runtime config optional and handle none config at enclave init URL: https://github.com/apache/incubator-teaclave/pull/172 This is an automated message from the Apache Git Service

[GitHub] [incubator-teaclave] z1queue opened a new issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST

2019-12-17 Thread GitBox
z1queue opened a new issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST URL: https://github.com/apache/incubator-teaclave/issues/175 When i use mesatee for gbdt train, some hours later, it shows the ERROR log. https://user-images.githubusercontent.com/53327024/71055

[GitHub] [incubator-teaclave] mssun commented on issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST

2019-12-17 Thread GitBox
mssun commented on issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST URL: https://github.com/apache/incubator-teaclave/issues/175#issuecomment-566865445 Thanks for the report. We didn't handle the power transition issue right now. I found a sample code of how to han

[GitHub] [incubator-teaclave] z1queue commented on issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST

2019-12-17 Thread GitBox
z1queue commented on issue #175: ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST URL: https://github.com/apache/incubator-teaclave/issues/175#issuecomment-566868694 > Thanks for the report. We didn't handle the power transition issue right now. I found a sample code of how to

[GitHub] [incubator-teaclave] mssun opened a new pull request #176: Fix compilation issue

2019-12-17 Thread GitBox
mssun opened a new pull request #176: Fix compilation issue URL: https://github.com/apache/incubator-teaclave/pull/176 ## Description Fix compilation issue introduce by 4220e33 CI: https://ci.mesalock-linux.org/mssun/incubator-mesatee/175 --

[GitHub] [incubator-teaclave] mssun merged pull request #176: Fix compilation issue

2019-12-17 Thread GitBox
mssun merged pull request #176: Fix compilation issue URL: https://github.com/apache/incubator-teaclave/pull/176 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

[GitHub] [incubator-teaclave] m4sterchain opened a new pull request #177: Handle RPC invoke error

2019-12-17 Thread GitBox
m4sterchain opened a new pull request #177: Handle RPC invoke error URL: https://github.com/apache/incubator-teaclave/pull/177 ## Description Handle RPC invoke error. Fixes # (issue) Fix #85 ## Type of change (select applied and DELETE the others) - [ ] Bug fix

[GitHub] [incubator-teaclave] mssun closed issue #85: Improve error handling logic in current RPC.

2019-12-17 Thread GitBox
mssun closed issue #85: Improve error handling logic in current RPC. URL: https://github.com/apache/incubator-teaclave/issues/85 This is an automated message from the Apache Git Service. To respond to the message, please log

[GitHub] [incubator-teaclave] mssun merged pull request #177: Handle RPC invoke error

2019-12-17 Thread GitBox
mssun merged pull request #177: Handle RPC invoke error URL: https://github.com/apache/incubator-teaclave/pull/177 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub a

[GitHub] [incubator-teaclave] z1queue opened a new pull request #178: update how_to_build.md about export values

2019-12-18 Thread GitBox
z1queue opened a new pull request #178: update how_to_build.md about export values URL: https://github.com/apache/incubator-teaclave/pull/178 ## Description Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. F

[GitHub] [incubator-teaclave] mssun commented on issue #178: update how_to_build.md about export values

2019-12-18 Thread GitBox
mssun commented on issue #178: update how_to_build.md about export values URL: https://github.com/apache/incubator-teaclave/pull/178#issuecomment-566922517 Merged, thanks! This is an automated message from the Apache Git Serv

[GitHub] [incubator-teaclave] mssun merged pull request #178: update how_to_build.md about export values

2019-12-18 Thread GitBox
mssun merged pull request #178: update how_to_build.md about export values URL: https://github.com/apache/incubator-teaclave/pull/178 This is an automated message from the Apache Git Service. To respond to the message, please

[GitHub] [incubator-teaclave] z1queue opened a new pull request #179: handle the power transition for SGX_ERROR_ENCLAVE_LOST

2019-12-18 Thread GitBox
z1queue opened a new pull request #179: handle the power transition for SGX_ERROR_ENCLAVE_LOST URL: https://github.com/apache/incubator-teaclave/pull/179 ## Description Fix the ecall_ipc_entry_point, app sgx_error::SGX_ERROR_ENCLAVE_LOST #175 Fixes # (issue) ## Type of cha

[GitHub] [incubator-teaclave] mssun commented on issue #179: handle the power transition for SGX_ERROR_ENCLAVE_LOST

2019-12-18 Thread GitBox
mssun commented on issue #179: handle the power transition for SGX_ERROR_ENCLAVE_LOST URL: https://github.com/apache/incubator-teaclave/pull/179#issuecomment-566939810 Thank you for the PR. We will review it as soon as possible. BTW, how did you test this issue? I found that the sa

[GitHub] [incubator-teaclave] mssun opened a new pull request #180: Introduce proto_gen

2019-12-18 Thread GitBox
mssun opened a new pull request #180: Introduce proto_gen URL: https://github.com/apache/incubator-teaclave/pull/180 ## Description Introduce `proto_gen` to help us generate proto files. I remove the customized base64 serializer/deserializer because it will makes the issue mor

[GitHub] [incubator-teaclave] mssun opened a new issue #181: Investigate how to make cargo run in our generator offline (using vendored crates)

2019-12-18 Thread GitBox
mssun opened a new issue #181: Investigate how to make cargo run in our generator offline (using vendored crates) URL: https://github.com/apache/incubator-teaclave/issues/181 Hi, current `cargo run` in our generators do not use our vendored crates. We need to investigate how to fix this.

[GitHub] [incubator-teaclave] mssun opened a new pull request #182: Reorganize mesate_core/config and teaclave_config

2019-12-18 Thread GitBox
mssun opened a new pull request #182: Reorganize mesate_core/config and teaclave_config URL: https://github.com/apache/incubator-teaclave/pull/182 ## Description Reorganize `mesate_core/config` and `teaclave_config`. ## Type of change (select applied and DELETE the others)

[GitHub] [incubator-teaclave] dingelish opened a new pull request #183: Upgrade to teaclave-sgx-sdk v1.1.0

2019-12-19 Thread GitBox
dingelish opened a new pull request #183: Upgrade to teaclave-sgx-sdk v1.1.0 URL: https://github.com/apache/incubator-teaclave/pull/183 ## Description This PR is to upgrade teaclave-sgx-sdk v1.1.0 and Intel SGX SDK 2.7.1. Everything is fixed and test passed on my workstation. Needs h

[GitHub] [incubator-teaclave] dingelish opened a new pull request #183: Upgrade to teaclave-sgx-sdk v1.1.0

2019-12-19 Thread GitBox
dingelish opened a new pull request #183: Upgrade to teaclave-sgx-sdk v1.1.0 URL: https://github.com/apache/incubator-teaclave/pull/183 ## Description This PR is to upgrade teaclave-sgx-sdk v1.1.0 and Intel SGX SDK 2.7.1. Everything is fixed and test passed on my workstation. Needs h

[GitHub] [incubator-teaclave] dingelish opened a new pull request #184: Upgrade to SGX SDK v1.1.0

2019-12-19 Thread GitBox
dingelish opened a new pull request #184: Upgrade to SGX SDK v1.1.0 URL: https://github.com/apache/incubator-teaclave/pull/184 ## Description This PR is to upgrade teaclave-sgx-sdk v1.1.0 and Intel SGX SDK 2.7.1. Everything is fixed and test passed on my workstation. Fixes # (

[GitHub] [incubator-teaclave] mssun opened a new pull request #185: [docker] Fix docker building issue

2019-12-19 Thread GitBox
mssun opened a new pull request #185: [docker] Fix docker building issue URL: https://github.com/apache/incubator-teaclave/pull/185 ## Description Fix ubuntu 16.04 runtime docker building issue. I also remove the default `RUST_LOG` env, you can set when launching with the `docker-com

[GitHub] [incubator-teaclave] mssun opened a new pull request #186: Rewrite quickstart example command line interface

2019-12-19 Thread GitBox
mssun opened a new pull request #186: Rewrite quickstart example command line interface URL: https://github.com/apache/incubator-teaclave/pull/186 ## Description Two sub-commands: echo: ``` ./release/examples/quickstart echo -e release/examples/enclave_info.toml -i "Hel

[GitHub] [incubator-teaclave] mssun opened a new pull request #187: Rewrite README.md

2019-12-19 Thread GitBox
mssun opened a new pull request #187: Rewrite README.md URL: https://github.com/apache/incubator-teaclave/pull/187 ## Description Rewrite project `README.md`. Abandon previous on with a lot of fuzzy words. The rewrite is the first version which only contains necessary information. O

[GitHub] [incubator-teaclave] mssun opened a new pull request #188: Add asf.yaml to setup description of github repo

2019-12-20 Thread GitBox
mssun opened a new pull request #188: Add asf.yaml to setup description of github repo URL: https://github.com/apache/incubator-teaclave/pull/188 ## Description Use `.asf.yaml` to setup github repo. (refer to: https://cwiki.apache.org/confluence/display/INFRA/.asf.yaml+features+for+

[GitHub] [incubator-teaclave-sgx-sdk] mssun opened a new issue #184: `set_enclave_path` always return Ok(())

2019-12-20 Thread GitBox
mssun opened a new issue #184: `set_enclave_path` always return Ok(()) URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/184 I found that the `set_enclave_path` function always return `Ok(())`. I suggest this function should check whether the `path` exits or not and return a

[GitHub] [incubator-teaclave-sgx-sdk] dingelish commented on issue #184: `set_enclave_path` always returns Ok(())

2019-12-20 Thread GitBox
dingelish commented on issue #184: `set_enclave_path` always returns Ok(()) URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/184#issuecomment-568107746 @volcano0dr can we do this? This is an automated message

[GitHub] [incubator-teaclave-sgx-sdk] dingelish closed issue #183: 关于远程证实

2019-12-20 Thread GitBox
dingelish closed issue #183: 关于远程证实 URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/183 This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the

[GitHub] [incubator-teaclave] litongxin1991 commented on issue #138: [RFC][Do Not Merge] RPC protocols

2019-12-20 Thread GitBox
litongxin1991 commented on issue #138: [RFC][Do Not Merge] RPC protocols URL: https://github.com/apache/incubator-teaclave/pull/138#issuecomment-568115612 # Description for Task protocols This is a draft protocol for Task. Task defines a computation process, that means, the comput

[GitHub] [incubator-teaclave] mssun merged pull request #188: Add asf.yaml to setup description of github repo

2019-12-20 Thread GitBox
mssun merged pull request #188: Add asf.yaml to setup description of github repo URL: https://github.com/apache/incubator-teaclave/pull/188 This is an automated message from the Apache Git Service. To respond to the message,

[GitHub] [incubator-teaclave] rdzhou opened a new pull request #189: Generate *_enclave.so instead of *.enclave.so

2019-12-20 Thread GitBox
rdzhou opened a new pull request #189: Generate *_enclave.so instead of *.enclave.so URL: https://github.com/apache/incubator-teaclave/pull/189 ## Description - Generate *_enclave.so instead of *.enclave.so for the convenience of symbol retrieving. - Remove unused env MESATEE_TEST_MO

[GitHub] [incubator-teaclave] mssun merged pull request #189: Generate *_enclave.so instead of *.enclave.so

2019-12-20 Thread GitBox
mssun merged pull request #189: Generate *_enclave.so instead of *.enclave.so URL: https://github.com/apache/incubator-teaclave/pull/189 This is an automated message from the Apache Git Service. To respond to the message, ple

[GitHub] [incubator-teaclave-sgx-sdk] brenzi opened a new issue #185: cannot transmute between types of different sizes, or dependently-sized types

2019-12-22 Thread GitBox
brenzi opened a new issue #185: cannot transmute between types of different sizes, or dependently-sized types URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/185 I'm trying to build the `sgx-types` dependency for [my project](https://github.com/encointer/encointer-app). I

[GitHub] [incubator-teaclave-sgx-sdk] dingelish commented on issue #185: cannot transmute between types of different sizes, or dependently-sized types

2019-12-22 Thread GitBox
dingelish commented on issue #185: cannot transmute between types of different sizes, or dependently-sized types URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/185#issuecomment-568291829 Is this [rust isssue #61956](https://github.com/rust-lang/rust/issues/61956#issuecom

[GitHub] [incubator-teaclave-sgx-sdk] brenzi commented on issue #185: cannot transmute between types of different sizes, or dependently-sized types

2019-12-22 Thread GitBox
brenzi commented on issue #185: cannot transmute between types of different sizes, or dependently-sized types URL: https://github.com/apache/incubator-teaclave-sgx-sdk/issues/185#issuecomment-568297149 don't know. anyway, I found a workaround. The sgx-types dependency can be avoided for m

  1   2   3   4   5   6   >