add todo suggestions

Cargo.toml

@@ -70,6 +70,7 @@ tonic-build = "0.12"
 
 [features]
 default = []
+# TOOD: rename to testing and reverse uses in the code
 occlum = []
 reqwest = ["dep:reqwest"]
 actix-web = ["dep:actix-web", "actix-service", "actix-http"]

build.rs

@@ -1,9 +1,11 @@
 fn main() {
+    // TODO: make this conditional on occlum feature
     // TODO: should be conditional on the target platform (musl vs glibc)
     println!("cargo:rustc-link-search=/opt/occlum/toolchains/dcap_lib/musl");
     // Cargo will automatically know it must look for `libocclum_dcap.a`
     println!("cargo:rustc-link-lib=occlum_dcap");
 
+    // TODO: make this conditional if compiling grpcs examples
     tonic_build::configure()
         .build_server(true)
         .compile(&["examples/echo.proto"], &["examples"])

build_client.sh

@@ -34,6 +34,7 @@ occlum init && rm -rf image
 cp ../Occlum.json ./
 copy_bom -f ../client.yaml --root image --include-dir /opt/occlum/etc/template
 # TODO: "--enable-edmm Y" must be only for platforms that support SGX2
+# TODO: make sure the bundle needs SGX2 to run since SGX1 is vulnerable to https://x.com/PratyushRT/status/1828183761055330373
 occlum build --sign-key ../examples/signing_key.pem
 occlum package client.tar.gz